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

feat : optimizedPhoto #84

Merged
merged 2 commits into from
Dec 13, 2024
Merged

feat : optimizedPhoto #84

merged 2 commits into from
Dec 13, 2024

Conversation

Minsoek96
Copy link
Contributor

Image Optimization 성능 비교

커스텀 압축 구현과 brower-image-compression 압축의 성능을 테스트한 결과

성능 테스트 결과

이미지 크기 압축 방식 처리 시간 압축 후 압축률
9.5MB 브라우저 1707.80ms 3033.06KB 68.27%
커스텀 1231.70ms 167.31KB 98.25%
277.52KB 브라우저 229.10ms 237.02KB 14.60%
커스텀 60.90ms 18.64KB 93.28%
165.22KB 브라우저 237.70ms 138.45KB 16.20%
커스텀 146.30ms 11.15KB 93.25%
474.71KB 브라우저 205.80ms 402.01KB 15.32%
커스텀 137.30ms 46.81KB 90.14%

ISSUE

32312

avif, webp 등 일부 형식의 이미지 업로드시
network 400 error
sever - code: 504 , message : '이미지 파일만 업로드 가능합니다.'

임시 대응

모든 이미지 확장자를 JPEG로 변환 처리

  • webp, avif 파일의 경우: JPEG 변환 시 기존 압축률 30% 감소
  • 기타 이미지 파일: JPEG 변환 후 평균 압축률 50%

- 사진 최적화 기능 추가
- avif, webp 파일 업로드 시 네트워크 400에러 발생
- 업로드 제한 관련 서버 응답 코드 : 504
    - 에러 메시지 : "이미지 파일만 업로드 가능합니다."
@bunzzeok
Copy link
Contributor

@Minsoek96 덕춘! 고마워요 ~

근데 제가 잘 몰라서 그런거 같은데, webp 보다, jpeg 가 더 무겁지 않나요 ?

@bunzzeok
Copy link
Contributor

image

추가적으로 80번 issue와 동일한 문제 같음으로,
close #80

남겨주시면 issue close 가 될 거에요 !

@Minsoek96
Copy link
Contributor Author

@bunzzeok 브릭스가 말씀하신 게 맞아요!!!
다만 현재 서버에서 webp, avif 등 일부 확장자를 차단하는 문제가 있어, 캡슐 생성 시 정상적으로
동작하지 않는 문제 떄문에 일시적으로 JPEG 변환을 적용해 두었습니다.
서버 문제가 해결되면 converToWebP만 WebP로 수정하면 될 것 같아요.

그리고 앞서 언급한 압축률은 webp 변환 테스트 결과였는데, 그 과정에서 모든 업로드 파일이 webp로
변환하면서 위의 문제를 발견하게 되었어요. 제가 설명을 불명확하게 해서 오해의 소지가 있었네요. 미안합니다ㅎㅎ..

현재는 webp, avif파일의 경우 jpeg 변환하면 30% 정도 사이즈가 증가하지만 업로드는 정상적인걸로 확인했어요.
jpeg 에서 jpeg 같은 경우는 50% 정도 압축되는 것 같아요.!

@bunzzeok
Copy link
Contributor

@Gem-o-b 후니 이거 한번 확인해주세요 !

@bunzzeok
Copy link
Contributor

@Minsoek96 우선적으로 해당 부분은 백엔드가 완료될때까지 머지 할게요 !

80번 이슈는 완료해주시고, develop label 로 wepb 로 변환으로 이슈 하나 부탁드려요 ~ !

@bunzzeok bunzzeok merged commit f09a55b into uchumu:main Dec 13, 2024
1 check 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.

2 participants