merge: develop - #131
Merged
Merged
Conversation
이 줄이 main 에만 있어 develop 을 머지하면 사라지는 상태였다. 값이 없으면 애플리케이션 기본값인 https://heddy.example.com/s 가 쓰여 공유 링크가 존재하지 않는 도메인으로 나간다.
fix: develop 의 배포 compose 에 SHARE_PUBLIC_BASE_URL 복원
목록의 analysis_status 가 String 이라 생성된 스펙에 type: string 으로만 나갔다. 허용값 여섯 개는 스키마 설명 문구에만 있어 클라이언트가 눈으로 읽고 옮겨 적어야 했고, 값이 늘거나 이름이 바뀌어도 스펙은 그대로라 어긋난 것이 드러나지 않았다. 열거형을 이름으로 바꾸던 단계를 없애 허용값의 출처를 AnalysisJobStatus 하나로 둔다. 직렬화 결과는 이름 그대로라 응답 JSON 은 바뀌지 않는다.
기대값을 AnalysisJobStatus 에서 직접 뽑는다. 테스트에 여섯 값을 적어두면 그 목록이 두 번째 출처가 되어 애초에 없애려던 문제를 되풀이한다.
refactor: 분석 상태를 열거형 그대로 응답에 싣기
#127 을 고치면서 같은 자리에 설명 주석을 덧붙였는데, main 은 hotfix 로 같은 설정을 이미 추가해 둔 상태였다. 양쪽이 같은 줄에 각자 추가한 꼴이라 develop 을 main 으로 머지할 때 충돌이 난다. 설정값은 처음부터 동일했으므로 주석을 main 쪽에 맞춰 충돌 원인을 없앤다.
chore: 배포 compose 를 main 과 동일하게 맞춤
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
1 participant
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.
관련 이슈
변경 내용
analysis_status를 열거형으로 선언해 OpenAPI 스펙에enum으로 노출 ([Task] analysis_status 를 OpenAPI 스키마에 열거형으로 노출 #125)SHARE_PUBLIC_BASE_URL정리 ([Bug] develop 의 compose 에 SHARE_PUBLIC_BASE_URL 누락 — 머지 시 share_url 이 예시 도메인으로 회귀 #127)작업 목적
분석 상태의 허용값이 스키마 설명 문구에만 있어 클라이언트가 코드 생성으로 타입을 받지
못했다. 공유 링크 기본 도메인 설정은
main과develop이 서로 다른 형태로 들고 있어병합할 때마다 충돌하던 것을 한쪽으로 맞췄다.
테스트 방법
응답 JSON 은 바뀌지 않는다. 값을 직접 검증하는 기존 목록 API 통합 테스트가 수정 없이
그대로 통과하는 것이 근거다.
체크리스트
기타 사항
analysis_status의 직렬화 결과는 이전과 동일하다.SHARE_PUBLIC_BASE_URL은 아직 없다. 현재는 compose 기본값으로동작하며, 값을 환경 변수에 직접 넣어야 기본값 의존이 완전히 사라진다.