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

[BE] setting/s3-CloudFront 서비스 이전 #748

Merged
merged 10 commits into from
Dec 3, 2024
Merged

[BE] setting/s3-CloudFront 서비스 이전 #748

merged 10 commits into from
Dec 3, 2024

Conversation

ehtjsv2
Copy link
Contributor

@ehtjsv2 ehtjsv2 commented Dec 2, 2024

이슈

개발 사항

  • S3, CloudFront 생성
  • dev, prod환경변수 값 수정

전달 사항 (없으면 삭제해 주세요)

@ehtjsv2 ehtjsv2 added ⚙️ setting 환경 세팅 🖥 backend backend labels Dec 2, 2024
@ehtjsv2 ehtjsv2 self-assigned this Dec 2, 2024
Comment on lines -67 to +64
.key(KEY_PREFIX + newFilename)
.key(newFilename)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

원래 하나의 버킷에 우테코의 모든 프로젝트가 관리 되었어서 패키지로 분리했었습니다.(/fiendogly/..)
이제는 반갑개만 관리되니 key_prefix 삭제했습니다

Copy link

github-actions bot commented Dec 2, 2024

Test Results

226 tests   226 ✅  19s ⏱️
 46 suites    0 💤
 46 files      0 ❌

Results for commit c18eea9.

♻️ This comment has been updated with latest results.

Comment on lines 5 to 6
endpoint: https://d3obq7hxojfffa.cloudfront.net/
endpoint: https://d1m6m0er0dq0qo.cloudfront.net/
key-prefix: friendogly/
Copy link
Contributor

Choose a reason for hiding this comment

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

여기도 key-prefix 없어져야 하는게 맞나요?

Copy link
Contributor Author

@ehtjsv2 ehtjsv2 Dec 3, 2024

Choose a reason for hiding this comment

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

코드에서 없애서 영향은 안주지만, 오해할 수 있으니 없애겠습니다~!

Copy link
Contributor

@takoyakimchi takoyakimchi left a comment

Choose a reason for hiding this comment

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

👍

@ehtjsv2 ehtjsv2 merged commit 85e5177 into develop Dec 3, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖥 backend backend ⚙️ setting 환경 세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

S3, CloudFront AWS 계정 이전
3 participants