feat: 이미지 업로드 오브젝트 배열로 변경 (#24) #25
Conversation
Walkthrough이 변경 사항은 이미지 업로드 기능을 수정하여 단일 이미지 배열 처리에서 여러 필드를 지원하도록 전환합니다. Changes
Possibly related issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
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
Documentation and Community
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (1)
src/api/imageUpload/image.routes.ts (1)
30-36: 이미지 업로드 처리 로직이 깔끔합니다.카테고리별 처리가 효율적으로 구현되어 있습니다. 다만, 업로드 실패 시의 부분 성공 상태 처리가 필요할 수 있습니다.
다음과 같이 개별 파일 업로드 실패를 처리하는 것을 고려해보세요:
for (const [category, categoryFiles] of Object.entries(files)) { if (categoryFiles && categoryFiles.length > 0) { - const urls = await uploadImages(categoryFiles); - result[category as keyof typeof result] = urls; + try { + const urls = await uploadImages(categoryFiles); + result[category as keyof typeof result] = urls; + } catch (error) { + console.error(`${category} 이미지 업로드 실패:`, error); + result[category as keyof typeof result] = []; + } } }
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
src/api/imageUpload/image.routes.ts(2 hunks)
🔇 Additional comments (3)
src/api/imageUpload/image.routes.ts (3)
10-14: 필드 구성이 명확하고 적절합니다!
각 이미지 카테고리별로 최대 개수가 잘 정의되어 있습니다.
24-28: 응답 구조가 잘 정의되어 있습니다!
각 카테고리별 배열이 명확하게 초기화되어 있어 좋습니다.
48-67: API 사용 예시가 명확합니다!
curl 명령어와 응답 예시가 잘 문서화되어 있습니다.
| if (!req.files) { | ||
| res.status(400).json({ message: '이미지 파일이 없습니다.' }); | ||
| return; |
There was a problem hiding this comment.
🛠️ Refactor suggestion
입력값 검증을 강화하는 것이 좋겠습니다.
현재는 req.files의 존재 여부만 확인하고 있습니다. 각 필드별로 필수값 여부와 파일 형식을 검증하는 것이 좋겠습니다.
다음과 같이 검증 로직을 추가하는 것을 제안드립니다:
if (!req.files) {
res.status(400).json({ message: '이미지 파일이 없습니다.' });
return;
}
+
+ const files = req.files as { [fieldname: string]: Express.Multer.File[] };
+
+ // thumbnail은 필수값으로 처리
+ if (!files.thumbnail || files.thumbnail.length === 0) {
+ res.status(400).json({ message: '썸네일 이미지는 필수입니다.' });
+ return;
+ }
+
+ // 이미지 형식 검증
+ const allowedMimeTypes = ['image/jpeg', 'image/png', 'image/gif'];
+ for (const [category, categoryFiles] of Object.entries(files)) {
+ for (const file of categoryFiles) {
+ if (!allowedMimeTypes.includes(file.mimetype)) {
+ res.status(400).json({
+ message: `지원하지 않는 파일 형식입니다: ${file.originalname}`
+ });
+ return;
+ }
+ }
+ }📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| if (!req.files) { | |
| res.status(400).json({ message: '이미지 파일이 없습니다.' }); | |
| return; | |
| if (!req.files) { | |
| res.status(400).json({ message: '이미지 파일이 없습니다.' }); | |
| return; | |
| } | |
| const files = req.files as { [fieldname: string]: Express.Multer.File[] }; | |
| // thumbnail은 필수값으로 처리 | |
| if (!files.thumbnail || files.thumbnail.length === 0) { | |
| res.status(400).json({ message: '썸네일 이미지는 필수입니다.' }); | |
| return; | |
| } | |
| // 이미지 형식 검증 | |
| const allowedMimeTypes = ['image/jpeg', 'image/png', 'image/gif']; | |
| for (const [category, categoryFiles] of Object.entries(files)) { | |
| for (const file of categoryFiles) { | |
| if (!allowedMimeTypes.includes(file.mimetype)) { | |
| res.status(400).json({ | |
| message: `지원하지 않는 파일 형식입니다: ${file.originalname}` | |
| }); | |
| return; | |
| } | |
| } | |
| } |
Summary by CodeRabbit
새로운 기능
버그 수정
문서화