Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughAWS S3를 통한 상점 프로필 이미지 업로드 기능이 도입되었습니다. 이를 위해 S3 클라이언트 및 업로더 유틸리티, 환경설정, 예외 메시지, multipart/form-data 요청 처리, 엔티티 및 DTO 구조 변경이 반영되었습니다. 상점 생성 시 이미지 파일을 선택적으로 업로드할 수 있습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant UserController
participant UserService
participant S3Uploader
participant AmazonS3
participant MarketRepository
Client->>UserController: multipart/form-data로 상점 회원가입 요청 (JSON + 이미지)
UserController->>UserService: signupMerchant(request, image)
alt 이미지 파일이 존재할 경우
UserService->>S3Uploader: upload(image, dirName, userEmail)
S3Uploader->>AmazonS3: putObject
S3Uploader-->>UserService: 업로드된 이미지 URL 반환
else 이미지 파일 미첨부
UserService-->>UserService: 이미지 URL null 처리
end
UserService->>MarketRepository: Market 엔티티 생성 (imageUrl 포함)
UserService-->>UserController: 가입 결과 반환
UserController-->>Client: 응답 반환
Possibly related issues
Possibly related PRs
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (4)
src/main/java/danji/danjiapi/global/config/AwsConfig.java (1)
24-29: S3 클라이언트 생성 로직에 예외 처리를 추가하세요.S3 클라이언트 생성 시 잘못된 자격증명이나 리전 설정으로 인한 예외가 발생할 수 있습니다. 초기화 시점에서 연결 테스트나 예외 처리를 추가하는 것을 고려해보세요.
src/main/java/danji/danjiapi/domain/user/service/UserService.java (1)
42-44: S3 업로드 실패 시 사용자 경험 개선을 고려해보세요.현재 S3 업로드가 실패하면 전체 회원가입 프로세스가 실패합니다. 이미지 업로드 실패 시에도 회원가입은 성공하고 이미지만 별도로 재시도할 수 있는 방안을 고려해보세요.
src/main/java/danji/danjiapi/global/util/S3Uploader.java (1)
39-41: 예외 처리 시 로깅을 추가하세요.IOException이 발생했을 때 원본 예외 정보가 손실됩니다. 디버깅과 모니터링을 위해 로깅을 추가해보세요.
} catch (IOException e) { + log.error("S3 업로드 실패: bucket={}, filename={}", bucket, filename, e); throw new CustomException(ErrorMessage.USER_IMAGE_UPLOAD_FAILED); }src/main/java/danji/danjiapi/domain/user/controller/UserController.java (1)
39-40: 이미지 파일 유효성 검증을 추가하는 것을 고려해보세요.컨트롤러 레벨에서 이미지 파일에 대한 기본적인 유효성 검증(파일 크기, 타입 등)을 추가하면 더 안전한 API가 될 것입니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (10)
build.gradle(1 hunks)src/main/java/danji/danjiapi/domain/market/entity/Market.java(1 hunks)src/main/java/danji/danjiapi/domain/user/controller/UserController.java(2 hunks)src/main/java/danji/danjiapi/domain/user/dto/request/UserCreateMerchantRequest.java(1 hunks)src/main/java/danji/danjiapi/domain/user/service/UserService.java(2 hunks)src/main/java/danji/danjiapi/global/annotation/MultipartJsonRequest.java(1 hunks)src/main/java/danji/danjiapi/global/config/AwsConfig.java(1 hunks)src/main/java/danji/danjiapi/global/exception/ErrorMessage.java(1 hunks)src/main/java/danji/danjiapi/global/util/S3Uploader.java(1 hunks)src/main/resources/application.yml(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
src/main/java/danji/danjiapi/domain/user/dto/request/UserCreateMerchantRequest.java (1)
Learnt from: mingdodev
PR: mingdodev/danji-api#7
File: src/main/java/danji/danjiapi/domain/product/service/ProductService.java:23-32
Timestamp: 2025-07-06T10:15:26.201Z
Learning: danji-api 프로젝트의 데이터 스키마에서는 사용자당 하나의 마켓만 존재하도록 설계되어 있어서 MarketRepository.findByUserId()를 사용하는 것이 적절하다.
🧬 Code Graph Analysis (2)
src/main/java/danji/danjiapi/global/util/S3Uploader.java (1)
src/main/java/danji/danjiapi/global/exception/CustomException.java (1)
CustomException(3-14)
src/main/java/danji/danjiapi/domain/user/controller/UserController.java (1)
src/main/java/danji/danjiapi/domain/auth/controller/AuthController.java (1)
RestController(16-28)
🔇 Additional comments (8)
build.gradle (1)
36-39: AWS 종속성 최신 버전 및 보안 확인 완료Spring Cloud AWS 스타터(2.4.4)와 ion-java(1.11.10)는 Maven Central에서 확인된 최신 버전이며, GitHub 보안 권고에도 취약점 보고가 없습니다. 추가 조치가 필요하지 않습니다.
src/main/resources/application.yml (1)
19-28: AWS 구성이 올바르게 추가되었습니다AWS 구성이 보안 모범 사례에 따라 올바르게 구현되었습니다:
- 민감한 정보(액세스 키, 시크릿 키 등)를 환경 변수로 외부화
- S3 버킷 및 CloudFront 도메인 설정 포함
- 서울 리전(ap-northeast-2) 사용
리전이 하드코딩되어 있는 것은 한국 서비스 특성상 적절해 보입니다.
src/main/java/danji/danjiapi/domain/user/dto/request/UserCreateMerchantRequest.java (1)
18-18: DTO 구조 개선이 적절합니다
marketImageUrl필드를 제거하고 이미지 업로드를 멀티파트 파일 업로드로 분리한 것은 좋은 설계 결정입니다. 이는 JSON 요청 데이터와 파일 데이터를 명확하게 구분하여 코드의 가독성과 유지보수성을 향상시킵니다.src/main/java/danji/danjiapi/global/exception/ErrorMessage.java (1)
28-28: 이미지 업로드 실패 에러 메시지 추가가 적절합니다새로운 에러 메시지가 올바르게 추가되었습니다:
- SERVICE_UNAVAILABLE 상태 코드는 S3 업로드 실패와 같은 일시적 서비스 문제에 적합
- 한국어 메시지가 명확하고 사용자 친화적
- user 섹션에 논리적으로 배치됨
src/main/java/danji/danjiapi/domain/market/entity/Market.java (1)
43-44: imageUrl null 허용 동작 확인Market.create 메서드에서 imageUrl이 null로 전달될 경우에도
@builder가 해당 값을 그대로 설정하며, @column의 기본 nullable=true 제약으로 DB 저장 시 문제가 없습니다.
따라서 추가 조치 없이 변경사항을 승인합니다.src/main/java/danji/danjiapi/global/annotation/MultipartJsonRequest.java (1)
1-21: 잘 구현된 커스텀 어노테이션입니다.MultipartJsonRequest 어노테이션이 올바르게 구현되었습니다. Swagger 통합과 OpenAPI 문서화를 위한 메타데이터 설정이 적절합니다.
src/main/java/danji/danjiapi/domain/user/service/UserService.java (1)
38-44: 이미지 업로드 로직이 잘 구현되었습니다.조건부 이미지 업로드 로직이 적절하게 구현되어 있습니다. null 체크와 빈 파일 체크를 통해 안전한 처리가 되고 있습니다.
src/main/java/danji/danjiapi/domain/user/controller/UserController.java (1)
35-42: 멀티파트 요청 처리가 잘 구현되었습니다.@MultipartJsonRequest 어노테이션과 @RequestPart를 사용한 멀티파트 요청 처리가 적절하게 구현되었습니다. JSON 요청과 이미지 파일을 분리하여 처리하는 방식이 올바릅니다.
| public String upload(MultipartFile file, String dirName, String userEmail) { | ||
| String filename = getFilenameFromEmail(file, dirName, userEmail); | ||
|
|
||
| try { | ||
| ObjectMetadata metadata = new ObjectMetadata(); | ||
| metadata.setContentType(file.getContentType()); | ||
| metadata.setContentLength(file.getSize()); | ||
|
|
||
| amazonS3.putObject(new PutObjectRequest(bucket, filename, file.getInputStream(), metadata)); | ||
|
|
||
| return cloudFrontDomain + "/" + filename; | ||
|
|
||
| } catch (IOException e) { | ||
| throw new CustomException(ErrorMessage.USER_IMAGE_UPLOAD_FAILED); | ||
| } | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
파일 업로드 유효성 검증을 추가하세요.
현재 구현에서는 파일 타입, 크기, 내용에 대한 검증이 없습니다. 보안과 안정성을 위해 다음을 추가하는 것을 권장합니다:
- 허용된 파일 타입 검증 (예: 이미지 파일만)
- 파일 크기 제한
- 파일 내용 검증 (악성 파일 방지)
🤖 Prompt for AI Agents
In src/main/java/danji/danjiapi/global/util/S3Uploader.java between lines 27 and
42, the upload method lacks validation for the uploaded file. Add checks to
verify the file type against allowed image MIME types, enforce a maximum file
size limit, and implement content validation to detect potentially malicious
files before proceeding with the upload. If any validation fails, throw an
appropriate exception to prevent unsafe files from being uploaded.
- 특수문자 제거 및 중복 예방 #10
1 participant
Summary by CodeRabbit
신규 기능
버그 수정
문서화
환경설정