Skip to content

[Feat/#612] 이미지 검증 및 스웨거 가드 추가 - #613

Merged
songwol38 merged 2 commits into
developfrom
feature/issue-612
Nov 11, 2026
Merged

songwol38 merged 2 commits into
developfrom
feature/issue-612

Conversation

@songwol38

Copy link
Copy Markdown
Member

📌 관련 이슈번호

📌 PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • 리팩토링

📌 PR 요약

자료·과제 첨부에 공통 업로드 검증을 적용하고, 지원하지 않는 확장자와 용량 초과 등 업로드 오류를 일관된 응답으로 반환합니다.

📌 작업 세부 내용

  • 프로필은 이미지 확장자만, 일반 첨부는 이미지·PDF·Office·한글 문서·ZIP 확장자를 허용합니다.
  • 확장자는 대소문자를 구분하지 않으며, 파일당 최대 100MiB 제한을 적용합니다.
  • Multer 필터와 uploadToS3() 호출 시 확장자를 검증합니다.
  • 자료·과제 라우트에서 공통 업로드 미들웨어를 사용하도록 변경합니다.
  • 업로드 오류에 HTTP 상태와 오류 코드를 반환하고, 확장자 오류 응답에는 허용 목록을 제공합니다.
  • 공통 미들웨어 적용에 맞춰 테스트 mock을 보완합니다.

🔗 참고

확장자 기반 검증이며, 파일 내용과 확장자의 일치 여부 및 악성 파일 검사는 포함하지 않습니다.

@songwol38 songwol38 self-assigned this Nov 11, 2026
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Explain Change →

Caution

Review failed

The pull request is closed.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@songwol38
songwol38 merged commit 91d2b38 into develop Nov 11, 2026
2 of 3 checks passed
@songwol38
songwol38 deleted the feature/issue-612 branch November 11, 2026 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] 파일 업로드 검증 및 Swagger 접속 가드 설정

1 participant