이미지 등록 v1(multipart) 경로 삭제 - #969
Open
m-a-king wants to merge 1 commit into
Open
Conversation
- 클라(웹)가 v2 로 전환을 마쳐 서버가 바이트를 받을 이유가 없어졌다. 앱은 react-native-webview 래퍼라 웹 배포와 함께 넘어가므로 스토어 배포를 기다리지 않는다 - 삭제 시점은 prod 실측으로 확정했다. 처음 확인 때는 v1 이 2일간 5회 호출되고 v2 는 0회라 보류했고, 클라 배포 후 재확인해 v2 호출과 v1 중단(마지막 2026-08-20 15:19 UTC)을 보고 진행했다 - v1 은 재시도마다 새 UUID raw key 를 만들어, 응답 유실 후 재시도가 중복 위시를 만들고 등록 한도도 이중 차감했다. v2 는 imageKey 가 요청의 자연 키라 claim 이 한쪽만 등록한다 - ImagePresignService.deleteRawsQuietly 도 함께 삭제했다. v1 전용이라 호출부가 통째로 사라졌고, v2 는 raw 를 클라가 올리므로 서버가 회수하지 않고 폴링·lifecycle 에 맡긴다 - v1 테스트 중 v2 에 없던 검증 5개(다건 READY 전이·이벤트 1회 발행·CLONE 403·정원 32 초과·정원 동시성)는 v2 경로로 이관했다 - multipart 전용 검증(파트 미전송·빈 바이트·raw 업로드와 회수)은 v2 에 개념이 없어 함께 소멸시켰다. 동시성 테스트의 raw 회수 단언 자리에는 거부된 요청의 pending 매핑이 롤백으로 남는지를 넣어 같은 성질을 지킨다
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Situation
imageKey가 그 요청의 자연 키라 claim 이 한쪽만 등록한다.Task
Action
*Api문서 블록,*ApiExamples분기, 서비스·영속화 메서드,uploadRaw.deleteRawsQuietly도 함께 삭제했다. 작업 중 v1 전용이라 호출부가 통째로 사라진다는 것이 드러났다. v2 는 raw 를 클라가 올리므로 서버가 회수하지 않고, 매이지 못한 raw 는 폴링이 pending 매핑을 정리한 뒤 S3 lifecycle 이 만료시킨다.PIKI_APPUA 였는데, 앱이react-native-webview래퍼라 부르던 주체가 앱 안의 웹이었다. 웹 배포와 함께 넘어가므로 스토어 배포를 기다릴 필요가 없었다.테스트
*Api의 summary 에 있는 "v2" 표기는 유지했다. 경로와 클라 호칭이 그대로라 문서 이름만 바꾸면 참조가 흔들린다.Result
uri라벨, nginx 접근 로그)으로 재확인할 수 있다.연관 이슈