Skip to content

이미지 주소 S3 버킷 이름, 클라우드 프론트 도메인 주소 지우기 - #193

Open
jinhokim98 wants to merge 3 commits into
develop-260222from
feature/#189
Open

이미지 주소 S3 버킷 이름, 클라우드 프론트 도메인 주소 지우기#193
jinhokim98 wants to merge 3 commits into
develop-260222from
feature/#189

Conversation

@jinhokim98

@jinhokim98 jinhokim98 commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

issue

구현 사항

이미지 S3, 클라우드 프론트 주소 의존성을 제거했습니다.
저번 회의에서 storage.crew-wiki.site로 주소로 대체하기로 결정되어서 db에 현재 잔존하는 S3, 클라우드 프론트 주소가 모두 변경될 예정입니다. 이에 프론트에서도 이미 저장된 주소를 변환해 줄 필요가 없어져 대응 로직을 제거합니다.

그리고 CDN을 STATIC이라는 용어로 변경했습니다.
CDN이란 의미는 사용자가 저장하는 이미지도 포함하는 느낌이라서 정적 파일이라는 STATIC으로 변경했어요

🫡 참고사항

NEXT_PUBLIC_IMAGE_DOMAIN
NEXT_PUBLIC_STATIC_DOMAIN

으로 관리합니다.

이미지 도메인은 사용자가 저장하는 이미지 주소
STATIC 도메인은 우리 정적 파일(아이콘, 크루위키 이미지)을 저장하는 주소

변경된 env는 별도로 공유드릴게요.
그리고 아직 인프라 작업은 아직 하지 않았습니다. 이건 나중에 칼리와 맞춰서 해야할 듯 해요

@jinhokim98 jinhokim98 added this to the v3.3.0 milestone Feb 18, 2026
@jinhokim98 jinhokim98 added ⚙️ feat feature 🛠️ Setting project setting labels Feb 18, 2026
@jinhokim98 jinhokim98 moved this to In Review in crew-wiki-7-FE Feb 18, 2026
@jinhokim98 jinhokim98 self-assigned this Feb 18, 2026

@ShinjungOh ShinjungOh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ENV 변경이 있는 만큼 모든 환경에서 다 변경사항이 적용됐는지 확인이 필요하겠네요. 깃허브의 setings > secrets and variables 에서도 환경변수 설정을 추가해야겠어요!
참고사항에서 말씀하신 인프라 작업은 어떤 작업인가요? 1/11 정기회의에서 나왔던 '토다리의 가비아 작업이 필요한 대체 도메인 이슈' 이걸 얘기하신걸까요?_?
그리고 인프라 작업이 아직이라고 하셨는데, 배포 순서를 어떻게 생각하고 계신지도 궁금합니다! (이 PR 내용의 배포 시점이라든가..)

const pathParts = url.pathname.split('/');
const filename = pathParts[pathParts.length - 1];

const processedFilename = filename.replace(/\.[^/.]+$/, '') + '.jpeg';

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

이 파일이 삭제되면서 .jpeg로 강제 변환되는 로직이 사라지는데 기존 이미지에 영향을 미치진 않을까요?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

오 이건 현재 s3 이미지 확장자들을 살펴봐야 할 것 같아요.

전에 확장자가 없는 파일들이 s3에 올라가서 jpeg로 넣어준거였어요. 그리고 업로드할 때 jpeg로 변환해서 올리기 때문에 이제 jpeg가 아닌 파일들이 없을거라 생각했어요.
변환이 안 먹혀서 jpeg가 아닌 파일이 s3에 있다면 문제가 될 것 같아요. s3 들어가서 확인해봐야겠어요

@jinhokim98

jinhokim98 commented Feb 20, 2026

Copy link
Copy Markdown
Contributor Author

맞아요 env 깃헙에도 반영해줘야합니다.

가비아 작업도 필요하고 그 이외의 작업이 더 필요해요. 우선 칼리와 제 일을 먼저 적으면

칼리의 일

본문 속 이미지 링크 s3주소 cloudfront 주소 들어가있는 것 storage.crew-wiki.site로 db 데이터 바꾸기

제 일

storage.crew-wiki.site로만 올리고 받게 고치기 (이전 형식 폴백 작업 없애주기)

여기에 추가로 들어가야할 인프라 작업이에요

인프라

storage.crew-wiki.site 서브도메인 등록 -> 가비아

s3 버킷 내 서브 디렉터리 생성

지금은 버킷에 바로 uuid 별 이미지들이 저장되어있습니다. 이 방식을 아래 구조로 바꾸려 합니다.

  • 버킷/이미지
  • 버킷/정적파일

cloudfront 라우팅 작업

s3 원본을 cloudfront가 라우팅을 하도록 작업해야합니다.

  • 버킷/이미지 -> 클프배포주소/images
  • 버킷/정적 파일 -> 클프배포주소/static

storage.crew-wiki.site 연결

클프배포주소를 위 도메인을 사용하도록 바꿔야합니다.

  • 버킷/이미지 -> storage.crew-wiki.site/images/
  • 버킷/정적 파일 -> storage.crew-wiki.site/static

@jinhokim98

Copy link
Copy Markdown
Contributor Author

아 이 pr 반영은 8기 시연 전까지는 하고 싶어요

@chosim-dvlpr chosim-dvlpr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

리뷰 달았습니다!! 고생하셨어욤

Comment on lines -6 to -8
NEXT_PUBLIC_IMAGE_S3_DOMAIN: string;
NEXT_PUBLIC_IMAGE_CLOUDFRONT_DOMAIN: string;
NEXT_PUBLIC_CDN_DOMAIN: string;

@chosim-dvlpr chosim-dvlpr Feb 21, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

환경변수 네이밍이 변경되거나 제거됨에 따라 dev 배포 스크립트도 변경되어야 할 것 같아요~

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

리뷰 반영이 늦었어요..ㅜ 꼼꼼하게 봐주셔서 감사합니다~
여기에도 반영했어요
0bc6aa7

@ShinjungOh ShinjungOh added the ⛔️ 머지금지 Do not merge label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚙️ feat feature ⛔️ 머지금지 Do not merge 🛠️ Setting project setting

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

3 participants