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: s3 image 업로드 로직 수정 #116

Merged
merged 2 commits into from
Apr 24, 2024
Merged

fix: s3 image 업로드 로직 수정 #116

merged 2 commits into from
Apr 24, 2024

Conversation

tomatozil
Copy link
Contributor

@tomatozil tomatozil commented Apr 24, 2024

노션 api 페이지 참고
s3 이미지 업로드는 preSignedUrl을 기반으로 진행됩니다.
업로드 주체는 클라이언트이고, s3 버킷에 접근 권한을 가진 서버에서 클라이언트에게 접근 url을 발급해줍니다.

기존과 달라진 점은 회원가입과 아바타 수정 api 에서 avatar 데이터를 boolean 값으로 받습니다.
true일때

  • User 데이터베이스의 avatar 값을 '고정 url + 유저 닉네임 + 날짜' string 값으로 업데이트 합니다.
  • preSignedUrl을 발급받아 string값을 응답 데이터로 줍니다.

false라면

  • 회원가입시에는 nickname만 업데이트하고, 수정시에는 avatar 값을 null로 업데이트합니다.
  • preSignedUrl이 null인 응답 데이터를 줍니다.

@tomatozil tomatozil merged commit 0a69708 into main Apr 24, 2024
@tomatozil tomatozil deleted the s3image branch April 24, 2024 14:38
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.

1 participant