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

[Bug] 상점 아이템 이미지 로드 실패 #1583

Open
yoouyeon opened this issue Dec 20, 2024 · 2 comments · May be fixed by #1589
Open

[Bug] 상점 아이템 이미지 로드 실패 #1583

yoouyeon opened this issue Dec 20, 2024 · 2 comments · May be fixed by #1589
Assignees
Labels
bug Something isn't working

Comments

@yoouyeon
Copy link
Member

Desc

상점 아이템 이미지 요청 응답이 모두 301로 실패해서 대체 이미지가 보입니다.

Reproduce

  1. /takgu/store 페이지 이동

Wanted

Screen_Shot 2024-12-20 21 54 22

Bug img / video

Etc

@yoouyeon yoouyeon added the bug Something isn't working label Dec 20, 2024
@nerdchanii
Copy link

nerdchanii commented Dec 23, 2024

버그재보해주셔서 감사합니다!:)

@AreSain님께 협업을 요청해 같이 확인했습니다.

확인결과 과거에 쓰던 버킷에 저장되어있는 이미지들이 not_found 이미지로 표시되고 있는 것을 확인했습니다.
해결하기 위해서 이미지를 다시 등록해야할 것 같아 이미지 재등록을 시도했습니다.

이과정에서 POST /pingpong/admin/items/{itemId} 로 보내는 요청이 실패하고 있는 것을 확인했습니다.
백엔드 컨트롤러 로직은 아래와 같습니다. ref

} else if (imgData.getContentType() == null || !imgData.getContentType().equals("image/jpeg")) {

백엔드에서 타입을 type: 'image/jpeg'로 equal을 검사하는데, 현재 보내는 api는 아래와 같이 타입을 넣고 있는걸 확인했습니다. 이부분은 @seyeon22222 님이 확인해 주셨어요.

new Blob([imgData], { type: 'image/takgu/jpeg' })

해당라인을 {type:'image/jpeg'} 로 수정하면 해결할 수 있을 것으로 보입니다.

@nerdchanii nerdchanii self-assigned this Dec 23, 2024
@nerdchanii
Copy link

'image/takgu/jpeg'로 mime 타입으로 사용하고 있는 곳들을 좀 더 찾았습니다. 이 부분들을 이번 이슈에서 같이 개선했으면 하는데 어떤가요? 일반적인 MIME 타입을 쓸 수 있도록 하면 좋을것 같아요. 백엔드에서 이 부분들을 실제로 'image/takgu/jpeg'로 받고 있는지 확인이 필요한 상황입니다.

filetype: 'image/takgu/jpeg',

new Blob([imgData], { type: 'image/takgu/jpeg' })

filetype: 'image/takgu/jpeg',

@nerdchanii nerdchanii linked a pull request Dec 26, 2024 that will close this issue
@nerdchanii nerdchanii linked a pull request Dec 26, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants