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

chore: changed image url prefix #218

Merged
merged 3 commits into from
Apr 25, 2024
Merged

chore: changed image url prefix #218

merged 3 commits into from
Apr 25, 2024

Conversation

kyY00n
Copy link
Member

@kyY00n kyY00n commented Apr 25, 2024

우선순위

D-0

변경사항

  • 계정을 옮기면서 s3 버킷도 새로 만들어주었어요.
    • 기존에 데이터베이스에 저장되어있는 이미지 url 이 s3 Object url 이었기 때문에 버킷을 변경하면서 아주 불편했습니다.
    • 그래서 cloudfront를 연결해 https 를 활성화해줬고, alternative domain을 지정해줬습니다. 앞으로는 버킷을 옮기더라도 DNS 레코드가 가리키는 값만 바꾸면 됩니다.
  • 버킷명, credentials 등 대부분 시크릿 변수에 있어 큰 변경은 없었어요.
  • 이 PR의 변경사항은 기존에 변경을 고려하지 못해 불가피하게 바꿔야하는 부분입니다.

유의사항

  • 업로드 테스트
  • 데이터베이스 url 변경

+) 이번에 prefix를 바꾸면서 어쩔 수 없이 db에 저장된 url의 prefix를 직접 업데이트 해주었어요. 이 방식이 불편하고 안전하지도 않아서 DB에는 s3 객체 키만 저장하고, 애플리케이션에서 prefix를 더해서 응답할까 잠깐 고민했는데요, 소셜로 가입한 회원들 중에 이미지 동의한 회원들의 프로필이미지 url 은 해당 소셜에서 제공한 url 이라 고민을 기각했습니다. 시험기간이라 시간도 많지 않고요.

  • 배포 전에 새로운 시크릿을 반영해줘야합니다.

참조

@kyY00n kyY00n requested a review from DongLee99 as a code owner April 25, 2024 10:18
@kyY00n kyY00n merged commit dfd5bce into main Apr 25, 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.

1 participant