Skip to content

fix(application): 카카오 거리 계산 실패 원인 기록 #399 - #446

Merged
kusuri12-09 merged 1 commit into
developfrom
fix/399-kakao-distance-failure-logging
Oct 17, 2026
Merged

kusuri12-09 merged 1 commit into
developfrom
fix/399-kakao-distance-failure-logging

Conversation

@kusuri12-09

Copy link
Copy Markdown
Member

Summary

  • 수험번호 발급 중 카카오 거리 계산 실패가 발생해도 원인을 구분할 수 없던 로그를 개선합니다.
  • 실패 단계·원인·HTTP 상태·경로 결과 코드·예외 종류만 기록하고 기존 503 응답은 유지합니다.

Related Issue

Root Cause

  • 거리 어댑터가 HTTP 오류·빈 검색 결과·경로 실패·파싱 및 네트워크 예외를 모두 동일한 도메인 예외로 치환하면서 원인을 버렸습니다.
  • 운영 수험번호 발급 요청에서 DISTANCE_SERVICE_UNAVAILABLE 응답을 재현했으나, 운영 장애의 세부 원인은 아직 확정되지 않았습니다.

Fix Description

  • 지원자 좌표 검색·학교 좌표 검색·경로 조회 단계를 구분합니다.
  • HTTP 실패·빈 검색·좌표 형식·경로 결과·거리 누락·JSON 파싱·시간 초과·네트워크 오류를 안전한 진단값으로 기록합니다.
  • 주소·좌표·API 키·URL·응답 원문·예외 메시지·원본 예외는 로그에 포함하지 않습니다.
  • 이미 기록한 도메인 예외는 다시 기록하지 않으며 학교 좌표 캐시는 유지합니다.

Testing

  • Reproduced before fix
  • Verified after fix
  • Test details:
    • 거리 어댑터 테스트 9개 통과(XML 기준 실패·오류 0개).
    • bazel --install_base=D:/entry/entrydsm-platform/tmp/bazel-install-kakao test //systems/application/application-administration-adapter-out:kakao_distance_test --test_output=errors
    • 로컬 모의 서버로 단계별 HTTP·JSON 오류, 빈 검색, 경로 결과 코드, 시간 초과, 민감정보 미기록, 503 계약 및 캐시 재사용을 검증했습니다.
    • Application 서버를 격리된 메모리 DB로 기동하고 실제 거리 빈을 호출했습니다. 공개 장소에서 학교까지 14,898m를 반환했고, 학교와 동일한 주소는 경로 결과 코드 104와 진단 로그를 기록했습니다.
    • 실제 호출 검증은 서버 내부 빈 직접 호출이며 수험번호 일괄 발급 전체 흐름을 검증한 것은 아닙니다.

Risk & Impact

  • Risk level: Low
  • Affected areas:
    • Application 서비스의 카카오 거리 조회 실패 로그와 테스트 설정.
    • 거리 계산·일괄 발급 정책 및 외부 오류 응답은 유지합니다.
    • 배포 후 Application 로그에서 Kakao distance failure를 검색해 실제 운영 장애 원인을 확인해야 합니다.

Checklist

  • No new regressions introduced
  • Edge cases considered
  • Logs / error handling updated if needed

@kusuri12-09
kusuri12-09 merged commit 5e66733 into develop Oct 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants