Skip to content

Refactor/#19 s3 이미지 url 저장 구조 개선 객체 키 기반 저장#22

Open
pykido wants to merge 9 commits intomainfrom
Refactor/#19-S3_이미지_URL_저장_구조_개선-객체_키_기반_저장

Hidden character warning

The head ref may contain hidden characters: "Refactor/#19-S3_\uc774\ubbf8\uc9c0_URL_\uc800\uc7a5_\uad6c\uc870_\uac1c\uc120-\uac1d\uccb4_\ud0a4_\uae30\ubc18_\uc800\uc7a5"
Open

Refactor/#19 s3 이미지 url 저장 구조 개선 객체 키 기반 저장#22
pykido wants to merge 9 commits intomainfrom
Refactor/#19-S3_이미지_URL_저장_구조_개선-객체_키_기반_저장

Conversation

@pykido
Copy link
Member

@pykido pykido commented Sep 13, 2025

🔥 관련 이슈


📝 작업 상세 설명

AWS 계정 이동으로 인한 S3 버킷명 변경과 함께 이미지 저장 구조를 근본적으로 개선했습니다! DB 부분만 우선 급히 조정한 이유는 운영 서버 DB를 빠르게 손 봐야하기 때문입니다!

‼️주요 변경 사항

  1. 이미지 URL 저장 방식 개선
  1. 게시글 이미지 구조 개선
    현재 운영 서버 DB의 article 테이블 내 articlebody 컬럼을 보시면 이미지가 포함된 경우 로 되어있는 경우가 많습니다.
    이는 기존에 S3 presigned url을 사용하지 않는 경우, 백엔드 단으로 요청이 올 때 일단 ?로 두고 이미지를 따로 article_image 테이블 내에서 저장하였기 때문입니다. 이를 개선하는 작업을 ArticleBodyImageMigration.java를 통해 진행하였습니다.
스크린샷 2025-09-13 오후 6 59 25 스크린샷 2025-09-13 오후 6 59 50
  1. 마이그레이션 인터페이스 구현
    이미지를 저장하는 경우는 총 3가지(게시글, 프로필 사진, 팀 사진)인데 이 모든 경우 객체키값만을 저장하도록 치환해주었습니다.

⭐ 리뷰 포인트

이후에 해야할 작업이 남아있습니다!

  1. 현재 이미지를 저장하는 경우 2가지(프로필 사진, 팀 사진) 모두 객체 키값만을 저장하도록 리팩토링해야함. 현재 한 작업은 데이터베이스 속 기존 데이터들을 손 봤을뿐
  2. 또한 응답을 보낼 때는 완전한 image url로 반환되도록 해야함.

@pykido pykido requested a review from jinwon1234 September 13, 2025 09:54
@pykido pykido self-assigned this Sep 13, 2025
@pykido pykido added the 🔧 refactor 리팩토링 label Sep 13, 2025
Copy link
Collaborator

@jinwon1234 jinwon1234 left a comment

Choose a reason for hiding this comment

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

리팩토링하느라 수고하셨습니다! 마이그레이션 코드 관련해서는 딱히 드릴 말씀이 없는 것 같습니다.
하나 걱정되는 것은 제가 이미지 관련 코드를 쭉 살펴봤는데, 회원 탈퇴, 팀 삭제, 게시글 삭제 모두 이미지 삭제를 DB에서만 진행하고 S3에서는 삭제를 안해주고 있네요..! 이 부분 고려해서 나중에 S3 고아 객체 관리까지 추가하면 될 것 같습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 refactor 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] S3 객체 이미지 URL - 버킷 이름 변경

2 participants