개요
S3Service의 upload 메서드에서 발견된 개선 사항들을 추후 리팩토링 시 적용해야 합니다.
개선 필요 사항
1. 리소스 누수 방지
file.getInputStream() 사용 후 스트림을 닫지 않아 리소스 누수 위험
- try-with-resources 구문으로 InputStream 자동 종료 처리 필요
2. 파일명 정제
- 원본 파일명에 슬래시(
/, \) 등 경로 구분자 포함 시 S3 키 구조 손상 가능
- 정규식을 사용한 안전하지 않은 문자 제거/치환 로직 추가
3. ContentType 기본값 설정
file.getContentType()이 null일 경우 처리
- 기본값으로
application/octet-stream 설정 권장
4. 빈 파일 검증
- 업로드 전 파일이 비어있는지 확인하는 유효성 검사 추가
관련 파일
src/main/java/inha/gdgoc/domain/resource/service/S3Service.java
참고 링크
요청자: @kaswhy
개요
S3Service의 upload 메서드에서 발견된 개선 사항들을 추후 리팩토링 시 적용해야 합니다.
개선 필요 사항
1. 리소스 누수 방지
file.getInputStream()사용 후 스트림을 닫지 않아 리소스 누수 위험2. 파일명 정제
/,\) 등 경로 구분자 포함 시 S3 키 구조 손상 가능3. ContentType 기본값 설정
file.getContentType()이 null일 경우 처리application/octet-stream설정 권장4. 빈 파일 검증
관련 파일
src/main/java/inha/gdgoc/domain/resource/service/S3Service.java참고 링크
요청자: @kaswhy