Skip to content

Conversation

@Guryss
Copy link
Member

@Guryss Guryss commented Jul 20, 2025

⭐️Issue


🌟Motivation

  • 이미지 압축 함수 compressImage 로직을 수정했습니다.

🌟Key Changes

AS-IS

기존 코드는 이미지 업로드 시 최대 이미지 용량을 0.25MB 이하로 제한하고 있으며, 이를 만족하지 못할 경우 빈 데이터를 반환하도록 처리하고 있습니다.
피드 수정 시, 이미 업로드된 이미지를 다시 불러오게 되는데, 이때 UIImage로 디코딩된 후 다시 압축을 시도하면 원본보다 압축 효율이 떨어져 용량이 다시 커질 수 있습니다. 따라서 압축된 이미지가 다시 0.25MB를 초과하게 되면서, 빈 데이터가 반환되고 서버에는 유효하지 않은 이미지가 전달되어 500 에러가 발생했습니다.

TO-BE

에러가 발생하지 않도록, 압축된 이미지가 0.25MB보다 크게 되더라도 가능한 가장 작게 압축된 데이터를 반환하여 서버 500 에러 방지하도록 했습니다.

🌟Simulation

잘 작동합니다.

🌟To Reviewer

피드 수정을 반복하면 이미지 압축이 반복적으로 진행되어 점차 이미지 화질이 저하가 발생합니다.. 흠 최대한 scale은 건들지 않고 quality만 줄여 이미지 사이즈를 줄여보고자 하는데 쉽지 않네여 ㅠ

🌟Reference


@Guryss Guryss marked this pull request as ready for review July 20, 2025 04:38
Copy link
Member

@ena-isme ena-isme left a comment

Choose a reason for hiding this comment

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

PR 잘 읽어보았숩니다,저도 더 공부한 다음 좋은 방법 의논해보아야겠네유 !

Copy link
Contributor

@Naknakk Naknakk left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

지금은 일단 넘어가더라도 추후에 확실한 해결책이 필요하겠네요.

이미 고려했을지 모르겠지만, 지금 생각해 보기로는 피드 수정에 진입할 때, DTO->ENTITY 변환 과정에서 서버에서 받은 이미지인 경우에는 isAlreadyCompressed = true 이런 프로퍼티를 추가하는 방법은 어떨까요? 이 프로퍼티를 참조 해서 서버에서 받은 이미지인 경우에는 압축 과정 없이 서버에서 받아온 data를 그대로 다시 보내도록 하는 거죠 ..!

@ena-isme ena-isme merged commit 3780d27 into Develop Jul 22, 2025
@ena-isme ena-isme deleted the Fix/#606 branch July 23, 2025 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 피드 수정 시 무조건 이미지 압축이 성공될 수 있도록 로직 수정

4 participants