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

[NB-193] 프로필 이미지 저장 형식을 CDN URL로 변경 #72

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

Aram-su
Copy link
Contributor

@Aram-su Aram-su commented Nov 12, 2024

개요

NB-193

  • 프로필 이미지 업로드 후 DB에 저장되는 URL을 S3 URL에서 CDN URL로 변경함.
  • UserService
    • S3AndCdnUrlConverter에서 사용될 수 있도록 uploadPath 시작에 "/" 추가
    • 프로필 이미지 업로드 성공 시 CDN URL로 변경 로직 추가
  • S3ImageUploader
    • 변경된 uploadPath에 맞게 로직 수정

PR 유형

어떤 변경 사항이 있나요?

  • 버그 수정
  • 코드 리팩토링

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고 (Ctrl + 클릭하세요.)
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

프로필 이미지 업로드 후 DB에 저장되는 URL을 S3 URL에서 CDN URL로 변경함.

주요 변경사항:
 - 프로필 이미지 URL을 S3 경로에서 CDN 경로로 변경

구현 상세:
 - UserService : S3AndCdnUrlConverter에서 사용될 수 있도록 uploadPath 시작에 "/" 추가, 프로필 이미지 업로드 성공 시 CDN URL로 변경 로직 추가
 - S3ImageUploader : 변경된 uploadPath에 맞게 로직 수정
Copy link
Contributor

@liljoon liljoon left a comment

Choose a reason for hiding this comment

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

프로필 이미지 CDN URL로 변경된 것 확인했습니다.

@Aram-su Aram-su merged commit 94d76a6 into dev Nov 12, 2024
1 check passed
@Aram-su Aram-su deleted the refactor/profile-image-update branch November 12, 2024 07:17
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