Repository navigation
feat(be): 주가·환율·벤치마크·뉴스 조회 API 분리 - #71
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
주가·환율·벤치마크·뉴스를 한 GET에서 기다리던 흐름을 네 조회 API로 나눕니다. 각 응답은 해당 자료와 수집 상태만 반환하므로 콘솔이 완료한 자료부터 표시하고 실패한 조회만 재시도할 수 있습니다.
/api/v1/market-dataGET은/prices,/exchange-rates,/benchmarks,/news로 교체합니다.검증: 로컬 Docker 단위 테스트 529개·골든 E2E 1개, 예외 없는 의존성 감사와 runtime 빌드 통과. 배포용 FE/BE 이미지와 격리 MySQL에 합성 시세·뉴스를 넣어 네 병렬 조회, 입력 거부, 기존 계산값 및 현금 1,500 보존을 확인했습니다. 실제 브라우저에서 네 GET을 503으로 만들고 하나씩 복구했으며, 각 재시도는 해당 GET만 1회 증가시키고 계산 POST는 전후 4건으로 같았습니다. 운영 DB·운영 자격증명은 사용하지 않았습니다.
배포 호환성: 옛 통합 GET을 호출하는 콘솔은 새 API와 호환되지 않습니다. 두 저장소의 운영 반영을 함께 계획하고 이전 탭을 새로고침해야 합니다. 구 버전 콘솔을 계속 지원하는 점진 배포가 필요하면 먼저 임시 호환 경로를 준비해야 합니다. 이 PR은 dev 대상이며 운영에는 아직 반영하지 않았습니다.