Merged
Conversation
- `fileName` 생성 방식을 `String.format` 대신 문자열 템플릿 사용 - `preSignedUrlExpirationTime` 생성 방식을 시간 계산 후 설정 방식 대신 생성자로 직접 설정하도록 변경
Closed
1 task
Test Results67 tests 67 ✅ 2s ⏱️ Results for commit 868976f. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✅ PR 유형
어떤 변경 사항이 있었나요?
✏️ 작업 내용
S3 버킷에 한글 파일명으로 업로드 시 발생하는 오류를 해결하였습니다.
Note
해당 오류가 발생한 원인이 무엇인가요?
metadata의contentDisposition은 브라우저가 파일 다운로드 시 처리하는 HTTP 헤더입니다. 해당 데이터는 영어 이외의 문자를 포함하면 반드시 인코딩을 적용해야 하는데, S3 버킷에 한글 파일을 업로드할 때 인코딩이 되는 로직이 빠져있어 발생한 오류입니다.SWAGGER Test
S3 버킷 업로드 정보
S3 버킷에서 다운로드하는 이미지
PreSignedUrl생성 코드를 간결화하고 가독성을 향상시켰습니다.S3 버킷 업로드 정보
🔗 관련 이슈
💡 추가 사항
PreSignedUrl을 통해 업로드되는 파일은UUID와 파일 이름을-을 통해서 구별하도록 변경하였습니다!