[PR] FIX: Issue #62

[oosuhada]

core exceptions 이슈 해결 내용

수정 대상

backend/app/core/exceptions.py

해결한 이슈

1. HTTPException headers 누락

기존 _http_exception_to_response()는 HTTPException 또는 StarletteHTTPException을 프로젝트 표준 JSON 응답으로 변환하면서 원본 예외의 headers를 전달하지 않았습니다.

그 결과 아래와 같은 HTTP 표준/프레임워크 헤더가 누락될 수 있었습니다.

401 WWW-Authenticate
405 Allow
429/503 Retry-After

수정 후에는 exc.headers를 JSONResponse에 함께 전달하도록 변경했습니다.

headers = getattr(exc, "headers", None)

return JSONResponse(
    status_code=exc.status_code,
    content=...,
    headers=headers,
)

2. 에러 봉투 형태 해석 불일치

기존 _build_http_exception_response()는 detail이 dict일 때 detail.get("error") 값을 그대로 error_code로 사용했습니다.

하지만 프로젝트 표준 에러 봉투는 error가 문자열이 아니라 아래와 같은 dict 구조입니다.

{
  "error": {
    "code": "INVALID_JOB_ID",
    "detail": "UUID 형식이 아닙니다.",
    "field": "job_id",
    "retryable": false
  }
}

이 경우 기존 로직은 error.code를 읽지 못해 에러 코드가 잘못 변환될 수 있었습니다.

수정 후에는 detail["error"]가 dict인 경우 내부의 code, detail, field, retryable 값을 우선 사용하도록 변경했습니다.

raw_error = detail.get("error")
if isinstance(raw_error, dict):
    error_code = raw_error.get("code") or _default_error_code(exc.status_code)
    error_detail = raw_error.get("detail") or detail.get("detail")
    field = raw_error.get("field") or detail.get("field")
    retryable = raw_error.get("retryable", detail.get("retryable"))

검증

문법 검증을 통과했습니다.

py_compile 통과
- backend/app/core/exceptions.py

참고

별도 테스트 파일은 추가하지 않고, 실제 이슈 해결 코드는 backend/app/core/exceptions.py에만 반영했습니다.

---

Retroactive merge-linked issues

• Closes #62

---

Originally authored by woovii000 as PR #71

[oosuhada]

[@woovii000 님의 코멘트]

리뷰 코멘트에서 언급해주신 전역 예외 응답 포맷 회귀 방지 테스트를 추가했습니다.

추가 테스트 파일:

• backend/tests/unit/test_core_exceptions.py

검증 케이스:

• HTTPException.headers가 표준 JSON 응답 변환 후에도 보존되는지 확인
  • 예: WWW-Authenticate
• 이미 표준 에러 봉투 형태의 detail이 들어온 경우에도 headers가 유지되는지 확인
  • 예: X-Trace-Id
• detail["error"]가 dict인 부분 에러 봉투 형태에서 내부 code, detail, field, retryable 값을 올바르게 읽는지 확인
  • 예: INVALID_JOB_ID, job_id, retryable=false

문법 검증은 통과했습니다.

python -m py_compile app/core/exceptions.py tests/unit/test_core_exceptions.py

[oosuhada]

[@KimHyo1 님의 코멘트]

#71 최신 head 5e5c51e 기준으로 확인했습니다.

HTTPException/StarletteHTTPException을 표준 JSON 응답으로 변환할 때 원본 headers를 JSONResponse로 넘기도록 보완되어 401/405/Retry-After 계열 헤더 누락 이슈가 해결됩니다.

또한 detail["error"]가 dict인 부분 에러 봉투 형태일 때 code, detail, field, retryable을 내부 값 기준으로 읽도록 수정되어, 기존처럼 error dict 전체가 문자열화되어 error.code로 들어갈 가능성도 정리됐습니다.

현재 PR diff 범위에서는 추가로 막을 만한 이슈 없습니다. 다만 이 파일은 전역 예외 응답 포맷을 담당하니, 다음 보완 작업에서 헤더 보존/부분 에러 봉투 변환 케이스를 단위 테스트로 고정해두면 더 안정적일 것 같습니다.

[oosuhada]

[@smmini 님의 코멘트]

코드 리뷰 — PR #71 fix/core-exceptions

종합 판정: ✅ APPROVE (승인)

HTTP 예외 처리기에서 발생하던 헤더 유실 오류 및 에러 봉투 중첩 딕셔너리 파싱 정합성 결함(Issue #62)을 우수하게 정비해 주셨습니다.

특히 다음 개선 사항들의 설계가 매우 훌륭합니다:

• 예외 헤더 전달 보장: _http_exception_to_response 내부에서 getattr(exc, "headers", None) 처리를 적용해 JSONResponse에 바인딩하여 복원함으로써, WWW-Authenticate나 Retry-After 등 주요 HTTP 헤더가 유실 없이 안정적으로 수신되는 구조가 확보되었습니다.
• 중첩 에러 파싱 유연화: raw_error가 딕셔너리 형태일 때 내부 필드들을 해석하도록 isinstance(raw_error, dict) 조건 분기를 가미하여 에러 응답 규격과의 호환성이 대폭 강화되었습니다.
• 테스트 통일성 유지: 신규 추가된 test_core_exceptions.py 역시 프로젝트 전체 테스트 컨벤션에 완전히 부합하게 unittest 스타일로 작성되었으며 3개의 단위 검증 케이스가 모두 정상 통과합니다.

전체 테스트 스위트(69 passed)에 어떠한 사이드 이펙트도 유발하지 않고 청정하게 빌드되므로 즉시 승인(Approve)합니다. 고생하셨습니다!

[oosuhada]

[@KimHyo1 님의 코멘트]

#71 최신 head 291c250 기준으로 재확인했습니다.

이전 승인 이후 추가된 backend/tests/unit/test_core_exceptions.py에서 HTTPException.headers 보존, 이미 표준 에러 봉투인 detail의 headers 보존, detail["error"]가 dict인 부분 봉투의 code/detail/field/retryable 변환을 각각 고정하고 있어요.

핵심 수정인 headers 전달과 중첩 error dict 파싱도 현재 diff에서 의도대로 유지되어 있습니다. 현재 PR diff 범위에서는 추가로 막을 만한 이슈 없습니다.