Skip to content

fix: CRC32 헤더 비활성화 설정 추가#46

Merged
YuuuuuuYu merged 1 commit intodevelopfrom
fix/upload-image-jpg
Feb 25, 2025
Merged

fix: CRC32 헤더 비활성화 설정 추가#46
YuuuuuuYu merged 1 commit intodevelopfrom
fix/upload-image-jpg

Conversation

@YuuuuuuYu
Copy link
Member

🚀 작업 내용 설명

  • S3Config 수정

📢 그 외

문제

  • R2 스토리지가 AWS S3 API와 호환되도록 지원하지만, 일부 호환이 안되는 문제들이 있음

원인

  • R2 스토리지는 x-amz-checksum-crc32 헤더에 대해 지원을 안함
  • 해당 헤더는 S3에 객체를 업로드할 때, 데이터 무결성 확인을 위해 사용하는 헤더임

해결방법

  • SDK가 필요할 때만 체크섬을 검증하도록 S3Client 설정 값 수정
    • requestChecksumCalculation
    • responseChecksumValidation

📌 관련 이슈

close #45

@YuuuuuuYu YuuuuuuYu requested a review from wlsh44 February 25, 2025 08:30
@YuuuuuuYu YuuuuuuYu self-assigned this Feb 25, 2025
Copy link
Contributor

@wlsh44 wlsh44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

R2 설정은 또 다른가보네요.. 고생하셨습니다!

@YuuuuuuYu
Copy link
Member Author

R2 설정은 또 다른가보네요.. 고생하셨습니다!

S3 SDK를 가져다가 쓴다는 느낌?인것 같은데
그만큼 완벽히 호환이 안됐던것 같아요 ㅋㅋㅋ

@YuuuuuuYu YuuuuuuYu merged commit 552efb1 into develop Feb 25, 2025
2 checks passed
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.

이미지 jpg 확장자 오류

2 participants