feat: ncp object storage 연동

[1117mg]

📌 PR 제목

• [FEATURE] NCP에 영상 업로드 기능 구현 #28

📋 작업 내용

• NCP Object Storage 연동 및 테스트
• 루트미션 등록, 수정, 삭제, 조회 API
• 도전영상 및 썸네일 업로드
• /api/init-data: 암장레벨, 브랜드 imageUrl 실제 값으로 변경
• route mission difficulty example값 변경 (ex. V3-> BLUE)

✅ 체크리스트

• 테스트를 완료했나요?
• 코드 컨벤션을 지켰나요?
• 관련 문서를 업데이트 했나요?

🚨 주의사항

• 리뷰어가 주의깊게 봐야할 부분이 있다면 적어주세요.

[claude]

Claude finished @1117mg's task —— View job

---

코드 리뷰 완료

변경 사항 요약

이번 PR은 NCP Object Storage 연동을 통한 비디오/이미지 업로드 기능과 루트미션 CRUD API를 구현했습니다. 주요 변경사항으로는 AWS S3 호환 API를 사용한 NCP 스토리지 연동, 청크 업로드를 통한 대용량 비디오 처리, 그리고 루트미션 관리 API가 추가되었습니다. 전반적으로 구현이 잘 되어있으나 몇 가지 보안 및 성능 이슈가 발견되었습니다.

코드 리뷰

보안 이슈

• src/main/resources/application.yaml:78-82 - issue: NCP 인증 정보가 평문으로 노출 - 환경변수는 사용했지만 버킷명과 엔드포인트는 하드코딩되어 있음, 민감한 정보는 모두 환경변수로 처리 필요

성능 및 리소스 관리

• src/main/java/com/climbup/climbup/attempt/upload/service/UploadServiceImpl.java:54-57 - suggestion: 예외 발생 시 임시 파일 정리가 누락 - try-catch-finally 블록으로 파일 정리 보장 필요
• src/main/java/com/climbup/climbup/route/service/RouteMissionServiceImpl.java:191-209 - issue: 임시 파일 정리 부족 - 예외 발생 시 temp 디렉토리 파일이 누적될 수 있음, finally 블록에서 정리 보장 필요

버그 가능성

• src/main/java/com/climbup/climbup/route/service/RouteMissionServiceImpl.java:196 - question: 파일명 중복 가능성 - System.currentTimeMillis()만으로는 동시 업로드 시 충돌 가능, UUID 추가 고려 필요

[claude]

Claude finished @1117mg's task —— View job

---

코드 리뷰 완료

변경 사항 요약

NCP Object Storage 연동을 통한 비디오/이미지 업로드 기능과 루트미션 CRUD API를 구현했습니다. AWS S3 호환 API를 사용하여 청크 업로드를 지원하고, 대용량 비디오 파일을 처리할 수 있도록 개선되었습니다. 전체적으로 구현이 잘 되어있으나 몇 가지 보안 및 리소스 관리 이슈가 발견되었습니다.

코드 리뷰

src/main/resources/application.yaml:80-82 - issue: NCP 버킷명과 엔드포인트가 하드코딩됨 - 민감한 정보는 모두 환경변수로 처리 필요

src/main/java/com/climbup/climbup/attempt/upload/service/UploadServiceImpl.java:59 - suggestion: 예외 발생 시 임시 파일이 삭제되지 않음 - try-finally 블록으로 파일 정리 보장 필요

src/main/java/com/climbup/climbup/attempt/upload/util/FileUtils.java:14 - question: System.currentTimeMillis()만으로는 동시 업로드 시 파일명 충돌 가능성 - UUID만 사용하거나 nanoTime() 추가 고려 필요

---