Skip to content
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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

nerdchanii
Copy link

📌 개요

  • [Bug] 상점 아이템 이미지 로드 실패 #1583 에서 이미지 로드가 실패하는 원인을 파악해본 결과 이미지들이 과거에 썼던 버킷주소에서 이미지를 받아오고 있어서 그런것으로 확인했습니다.
  • 이를 해결하기 위해서 이미지를 다시 업로드를 시도해보았고, 업로드 과정에서 에러가 발생하고 있는 것을 확인했습니다.
  • 에러 원인은 792a0bb 에서 image 경로를 image/takgu로 고치는 과정에서 mime 타입까지 수정된 것으로 보입니다.

💻 작업사항

  • 잘못된 mime type을 수정했습니다
  • 'application/json' 과 'image/jpg', 'image/jpeg'를 인라인스트링으로 관리하던 부분을 상수로 추출했습니다.

✅ 변경로직

  • inline에선 mime타입을 기재하던 부분들을 드러내고, MIME_TYPE을 공용상수로 뺴냈습니다.
// 변경전
{type: 'image/jepg'}
{type: 'application/json'}
// 변경후
{type: MIME_TYPE.JPEG}
{type: MIME_TYPE.JSON}

💡관련 Issue

- 잘못된 mime type을 수정
- 'application/json' 과 'image/jpg', 'image/jpeg'를 인라인스트링으로 관리하던 부분을 상수로 추출
@nerdchanii nerdchanii added the fix bug fix label Dec 26, 2024
@nerdchanii nerdchanii self-assigned this Dec 26, 2024
@nerdchanii nerdchanii linked an issue Dec 26, 2024 that may be closed by this pull request
Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

이미지 잘 로드되네요!👍
배포 전에 지금 올라가 있는 테스트용 이미지 대신 원래 상점 이미지로 수정도 부탁드립니다!!

constants/common/mimeType.ts Outdated Show resolved Hide resolved
Co-authored-by: Yoon Jeongyeon <jyeon.yoon59@gmail.com>
@nerdchanii
Copy link
Author

이미지 잘 로드되네요!👍 배포 전에 지금 올라가 있는 테스트용 이미지 대신 원래 상점 이미지로 수정도 부탁드립니다!!

이미지를 서버로 부터 원격으로 이미지의 버켓주소를 받아와서 올리고 있는 형태라 프로덕션에는 배포후에 버켓에 이미지를 다시 올려야 수정가능할 것 같습니다. 테스트 서버는 수정해서 올려두겠습니다!:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix bug fix
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[Bug] 상점 아이템 이미지 로드 실패
2 participants