Skip to content

Conversation

@leesewon00
Copy link
Member

관련 이슈

작업 내용

  • 파일 저장시 url을 제외한 path만 저장하도록 수정
  • 저장된 파일의 url prefix를 확인할 수 있는 api 개발

특이 사항

초기에는 반환시에 url prefix를 붙여서 반환하려고 시도하였지만 기존 코드상 이를 끼워넣는데 많은 불편함이 존재하였습니다.
이에 api를 새로 개발하고 적절한 url을 가져가서 붙여서 사용하는게 좋지않을까 생각되어 수정해보았습니다.

image

다음과같이 url prefix를 확인할 수 있으며 cloudfront로 변경시에도 해당 값만 가져가서 붙이면 되지 않을까 생각됩니다.
경로들은 application-secret.yml 파일에 추가하여 작업하였습니다.

앞으로 DB 데이터 수정, 프론트 코드 수정, s3 정책수정 등 다양한 부분을 고려하여 신중히 작업해야될 것 같습니다.
해당 pr 머지 진행 후, api를 활용하여 생성한 경로로 cloudFront를 통해 파일을 잘 불러오는지 확인하고,
DB 데이터를 수정(url prefix 제거), s3 정책 수정 순으로 진행하면 될 것 같습니다.

리뷰 요구사항 (선택)

Copy link
Member

@wibaek wibaek left a comment

Choose a reason for hiding this comment

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

lgtm

@wibaek
Copy link
Member

wibaek commented Sep 19, 2024

현재 일반 API 이용시 이미지를 바로 업로드하는 것이 아닌 <이미지 업로드 API로 업로드 -> 반환된 링크를 포함해서 제출> 단계로 진행되고 있는 부분이 url prefix 반환 구현등에 문제가 될 수 있을 것 같네요.
말씀하신대로 차츰 개발해 나가는 것이 좋을 것 같습니다 :)

@leesewon00 leesewon00 merged commit eeef801 into solid-connection:main Sep 19, 2024
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.

S3 파일 저장에 관한 수정, CloudFront 도입

2 participants