[보안] 내부 인증 토큰 미설정 시 fail-open 취약점 수정

[coderabbitai]

배경

PR #77 코드 리뷰 중 발견된 보안 이슈입니다.
요청자: @hwangtae123
관련 PR: #77
관련 코멘트: #77 (comment)

문제 내용

app/api/document.py Line 40에서 환경 변수 INTERNAL_API_TOKEN이 설정되지 않은 경우, 인증 없이 모든 요청이 통과됩니다 (fail-open).

if _INTERNAL_TOKEN and x_internal_token != _INTERNAL_TOKEN:
    raise HTTPException(status_code=403, detail="forbidden")

같은 파일 _ALLOWED_STORAGE_HOSTS에는 localhost, host.docker.internal 등 내부 호스트가 포함되어 있어, 운영 환경에서 env 하나만 누락되어도 내부 파일 fetch 트리거가 외부에 노출될 수 있습니다.

수정 방향

배포 시 아래 방향으로 수정:

• INTERNAL_API_TOKEN이 미설정된 경우 기본적으로 403 반환 (fail-closed)
• 로컬 개발 환경 우회는 별도 DEBUG 플래그(APP_ENV=development 등)로만 허용
• _INTERNAL_TOKEN이 설정된 경우 반드시 x_internal_token과 일치해야 통과

[moonhwoo]

@hwangtae123

[moonhwoo]

@coderabbitai 이거는 v1방식에서 쓰는건데 현재 실제에선 v2방식을 사용해서 internal_api_token설정 상관없을거 같은데 확인좀

[coderabbitai]

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Fetch issue details
gh issue view 79 --repo Project-Guideon/guideon-AI

echo "---"
# Search for internal_api_token usage
rg -rn "internal_api_token" --type py -C 3

echo "---"
# Search for v1/v2 API usage patterns
rg -rn "v1|v2" --type py -C 2 -l

echo "---"
# Find API route/client definitions
rg -rn "v2" --type py -C 3

Length of output: 25715

---

🏁 Script executed:

#!/bin/bash
echo "=== app/api/document.py ==="
cat app/api/document.py

echo ""
echo "=== app/api/qa.py (first 60 lines) ==="
head -n 60 app/api/qa.py

echo ""
echo "=== main.py or app entry point ==="
fd -e py "main" | head -5 | xargs cat

Length of output: 3788

---

@moonhwoo 확인해봤는데, 말씀하신 것과 반대 상황입니다.

app/api/document.py는 현재 실제로 사용 중인 엔드포인트입니다.

파일	엔드포인트	상태
app/api/document.py	POST /v1/documents/process	✅ 현재 활성 (Spring Core 연동, n 파이프라인)
app/api/document_legacy.py	POST /sites/{site_id}/documents/upload 등	⚠️ 레거시 (삭제 예정)

app/api/document_legacy.py 파일 상단 docstring에도 "Core 연동 완료 후 이 파일 삭제 예정" 이라고 명시되어 있고, 실제로 document_processor.py의 docstring에도 Spring Core → POST /v1/documents/process 흐름이 v2/n 파이프라인으로 기술되어 있습니다.

그리고 main.py에 두 라우터 모두 등록되어 있습니다:

app.include_router(document.router)         # 현재 활성 v2 엔드포인트
app.include_router(document_legacy.router)  # 레거시

따라서 이슈 #79의 INTERNAL_API_TOKEN fail-open 취약점은 현재 운영 중인 엔드포인트에 해당하며, 수정이 필요합니다. INTERNAL_API_TOKEN이 환경 변수에 미설정된 경우 모든 요청이 인증 없이 통과되는 상태입니다.

[hwangtae123]

@coderabbitai 스프링을 통해 검증된 사용자에 대해서만 문서 업로드를 받습니다.

[coderabbitai]

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
echo "=== app/api/document.py ==="
cat app/api/document.py

echo ""
echo "=== Search for auth/middleware/dependency files ==="
fd -e py | xargs grep -l "verify\|auth\|token\|depend" 2>/dev/null | head -10

echo ""
echo "=== app/core or app/middleware ==="
fd -e py -p "middleware|auth|depend" | xargs cat 2>/dev/null

echo ""
echo "=== Check verify_internal_token or similar dependency ==="
rg -rn "verify_internal\|internal_token\|INTERNAL_API_TOKEN\|Depends" --type py -C 5

echo ""
echo "=== main.py ==="
fd -e py "main" | head -3 | xargs cat

Length of output: 4077

---

@hwangtae123, 말씀해주신 내용 이해했습니다. Spring이 사용자 인증을 담당하고, 그 이후에 내부적으로 Python 서비스를 호출하는 구조군요.

다만 이슈 #79의 취약점은 사용자 인증이 아니라 서비스 간(Spring → Python) 인증에 관한 것입니다.

현재 코드 상태입니다:

_INTERNAL_TOKEN = os.getenv("INTERNAL_API_TOKEN", "")

if _INTERNAL_TOKEN and x_internal_token != _INTERNAL_TOKEN:
    raise HTTPException(status_code=403, detail="forbidden")

INTERNAL_API_TOKEN이 .env에 설정되지 않으면, Spring 이외의 어떤 호출자도 인증 없이 이 엔드포인트를 직접 호출할 수 있습니다.

만약 Python 서비스가 Docker 내부 네트워크에만 노출되어 있고 외부에서 직접 접근 불가능하다면, 실질적인 위험은 낮습니다. 그러나 포트가 외부에 열려 있거나 네트워크 설정이 변경될 경우 우회가 가능합니다.

권장 사항:

• 운영 환경에서 INTERNAL_API_TOKEN을 반드시 설정하여 방어 심층(defense-in-depth)을 유지하는 것을 권장드립니다.
• Python 서비스가 실제로 Docker 내부망에서만 접근 가능하다면, 이슈를 닫거나 우선순위를 낮춰도 무방합니다.

Python 서비스 포트가 외부에 노출되지 않도록 네트워크 구성이 보장되어 있으신가요?