fix: 업로드 URL 요청의 필수 필드와 validation 계약 정렬

[taejinn]

🔍 개요

• 협의한 A안에 따라 upload URL 요청의 세 필드를 서버 validation과 Java Swagger에서 필수로 보장합니다.

• close [공통] 업로드 URL 요청 필수 필드와 서버 검증 불일치 #2382

---

🚀 주요 변경 내용

• content_length에 @NotNull과 양수 검증을 적용합니다.
• content_type의 null·빈 문자열·공백을 거절합니다.
• file_name의 null·빈 문자열·공백을 거절합니다.
• 세 필드를 Java Swagger required로 표시합니다.
• 잘못된 요청은 400 INVALID_REQUEST_BODY로 반환합니다.

---

💬 참고 사항

• 영향
  • 심각도는 P2 요청 계약 오류이며 Web, Admin, Owner Web/Mobile, Android, Coop의 기존 non-null 요청 모델과 호환됩니다.
  • UploadUrlRequest의 validation과 required 문서 계약을 정렬합니다.
  • upload URL 발급 요청의 입력값 검증을 보장합니다.
• 검증
  • 세 필드 누락, content_length=0과 음수, blank content_type과 file_name의 표준 field error를 확인했습니다.
  • 정상 요청의 기존 response를 확인했으며 validation 경계 5개와 HTTP message converter 회귀 테스트 1개가 통과했습니다.

---

✅ Checklist (완료 조건)

• 코드 스타일 가이드 준수
• 테스트 코드 포함됨
• Reviewers / Assignees / Labels 지정 완료
• 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

[coderabbitai]

Warning

Review limit reached

Next included review available in 50 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 94771c4d-1777-4795-ada9-56274dc62513

📥 Commits

Reviewing files that changed from the base of the PR and between 2004995 and f6c51e8.

📒 Files selected for processing (2)

• src/main/java/in/koreatech/koin/infrastructure/s3/dto/UploadUrlRequest.java
• src/test/java/in/koreatech/koin/unit/infrastructure/s3/dto/UploadUrlRequestValidationTest.java

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

Unit Test Results

1 117 tests   1 114 ✔️  2m 32s ⏱️
   250 suites         3 💤
   250 files           0 ❌

Results for commit f6c51e8.

♻️ This comment has been updated with latest results.