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

refactor : 이미지 저장 시 이름 형태 수정 #225

Merged
merged 7 commits into from
Dec 6, 2024
Merged

Conversation

rladmstn
Copy link
Contributor

@rladmstn rladmstn commented Dec 5, 2024

📌 Related Issue

close #224

🚀 Description

  • user 프로필 사진과 group 사진 저장 시, 각각의 고유 정보를 포함하도록 수정했습니다.
  • 원래 있던 이미지와 동일하면 불필요한 s3 이미지 추가/삭제를 하지 않기 위해 이미지명으로 비교합니다.
  • UTF-8 encoding으로 해보니 s3에서 진행하는 encoding과 형식이 조금 달라, 기존에 갖고 있던 이미지를 decoding해서 비교하는 방식으로 진행했습니다.
  • 사진 요청에 null로 입력이 들어오면 프로필 사진은 null로 설정됩니다.
  • 테스트 환경의 s3와 실제 서버 s3가 분리되지 않아서 userId만으로는 구분이 안될 것 같아 user Email 정보를 포함한 상태입니다.
  • 하지만 이는 테스트로 인해 로직이 수정되기에 좋지 못한 설계입니다.
  • 이건 추후에 이미지 이름에 test prefix를 추가해서 테스트용 사진과 실제 사진을 구분할 수 있도록 수정해야하지 않을까 싶습니다.

📢 Review Point

📚Etc (선택)

@rladmstn rladmstn added new-feature 기능 추가 refactoring 리팩토링 labels Dec 5, 2024
@rladmstn rladmstn requested review from s-hwan and sh0723 December 5, 2024 15:00
@rladmstn rladmstn self-assigned this Dec 5, 2024
@sh0723 sh0723 merged commit 9fe68e3 into develop Dec 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature 기능 추가 refactoring 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor : 이미지 저장 시 이름 형태 수정
2 participants