-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Fix] MIME type 상수화 #1589
base: main
Are you sure you want to change the base?
The head ref may contain hidden characters: "Fix/#1583-bug-\uC0C1\uC810-\uC544\uC774\uD15C-\uC774\uBBF8\uC9C0-\uB85C\uB4DC-\uC2E4\uD328"
[Fix] MIME type 상수화 #1589
Conversation
- 잘못된 mime type을 수정 - 'application/json' 과 'image/jpg', 'image/jpeg'를 인라인스트링으로 관리하던 부분을 상수로 추출
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이미지 잘 로드되네요!👍
배포 전에 지금 올라가 있는 테스트용 이미지 대신 원래 상점 이미지로 수정도 부탁드립니다!!
Co-authored-by: Yoon Jeongyeon <jyeon.yoon59@gmail.com>
이미지를 서버로 부터 원격으로 이미지의 버켓주소를 받아와서 올리고 있는 형태라 프로덕션에는 배포후에 버켓에 이미지를 다시 올려야 수정가능할 것 같습니다. 테스트 서버는 수정해서 올려두겠습니다!:) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정사항들 모두 확인했습니다! MIME type 상수화하니 더욱 좋아진 것 같아요! LGTM👍
또한 아이템 상점의 이미지들이 불러와지지 않는 에러에 대해서 자세히 확인해 보았습니다.
상점 데이터를 받아 올 때, imageUrl도 가져오게 되는데 해당 imageUrl이 잘못되었다고 뜨네요.😅 NoSuchBucket
이라고 뜨는데 백엔드에서 버킷 이미지가 삭제되었거나 imageUrl을 잘못 반환하고 있는 오류로 생각됩니다. (이게 아니라면, 권한-검증 문제)
이 부분은 제가 백엔드분들과 협업 후 다시 체크해서 해결하도록 하겠습니다!
📌 개요
image/takgu
로 고치는 과정에서 mime 타입까지 수정된 것으로 보입니다.💻 작업사항
✅ 변경로직
💡관련 Issue