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

[BSVR-256] 환경변수 관리를 위한 AWS Secret Manger 도입 - V2 #187

Merged
merged 5 commits into from
Sep 9, 2024

Conversation

pminsung12
Copy link
Collaborator

📌 개요 (필수)

  • 환경변수 관리를 위한 AWS Secret Manger 도입 - V2

🔨 작업 사항 (필수)

  • io aws cloud starter 패키지 의존성 추가
  • 프로필별 secret manager 컨테이너 설정
  • yml 파일에 secret manager 적용
  • github action에 configure step 추가 및 환경 변수 삭제

⚡️ 관심 리뷰 (선택)

  • local에서는 secret manager 사용을 위해 aws cli를 세팅해주어야해요.
  • 컨플에 있는 값들을 .env로 저장해도 좋고 aws cli를 설치해 컨플에 있는 access key, secret key를 입력 후 region으로 ap-northeast-2를 입력하면 설정 끝!

🌱 연관 내용 (선택)

  • 관련 있는 내용, 또는 앞으로 고려할 내용을 적어주세요.

💻 실행 화면 (필수)

  • 잘 된다~

image

@pminsung12 pminsung12 added ✨ Feature 기능 개발 🛠 Setting 개발 환경 세팅 🚀 CI/CD CI/CD 관련 labels Sep 8, 2024
@pminsung12 pminsung12 self-assigned this Sep 8, 2024
@pminsung12 pminsung12 marked this pull request as ready for review September 8, 2024 21:30
@github-actions github-actions bot added the size/M label Sep 8, 2024
Copy link

github-actions bot commented Sep 8, 2024

Test Results

53 tests  ±0   53 ✅ ±0   4s ⏱️ ±0s
18 suites ±0    0 💤 ±0 
18 files   ±0    0 ❌ ±0 

Results for commit ec90026. ± Comparison against base commit 586fcbd.

Comment on lines +1 to +10
oauth:
kakaoClientId: ${KAKAO_CLIENT_ID}
kakaoAuthTokenUrlHost: ${KAKAO_AUTH_TOKEN_URL_HOST}
kakaoAuthUserUrlHost: ${KAKAO_AUTH_USER_URL_HOST}
kakaoRedirectUrl: ${KAKAO_REDIRECT_URL}
googleClientId: ${GOOGLE_CLIENT_ID}
googleClientSecret: ${GOOGLE_CLIENT_SECRET}
googleRedirectUrl: ${GOOGLE_REDIRECT_URL}
googleAuthTokenUrlHost: ${GOOGLE_AUTH_TOKEN_URL_HOST}
googleUserUrlHost: ${GOOGLE_AUTH_USER_URL_HOST}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이제 환경 변수들 중.앙.제.어. 들어갑니다~!

Copy link
Collaborator

Choose a reason for hiding this comment

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

🎉🎉🎉🎉🎉🎉🎉

@pminsung12
Copy link
Collaborator Author

이렇게 간단한 작업을... 괜히 sdk로 하려다가 4,5일을 삽질했네 😭 😭

Copy link
Member

@wjdwnsdnjs13 wjdwnsdnjs13 left a comment

Choose a reason for hiding this comment

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

확인 완료!
나는 .env에 변수 다 들어있는데 그냥 그대로 사용가능한거지??

Copy link
Collaborator

Choose a reason for hiding this comment

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

와ㅋㅋㅋㅋㅋ 스크립트 간결해지는 것 봐ㅋㅋㅋㅋㅋㅋ 속이 시원하구만 👍

Copy link
Collaborator

@EunjiShin EunjiShin left a comment

Choose a reason for hiding this comment

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

하반기 최고의 PR로 임명합니다 👍
스크린샷 2024-09-09 오후 11 08 46

@pminsung12
Copy link
Collaborator Author

확인 완료! 나는 .env에 변수 다 들어있는데 그냥 그대로 사용가능한거지??

엉 그렇게 사용해도돼!

@pminsung12 pminsung12 merged commit c024ffd into main Sep 9, 2024
5 checks passed
@pminsung12 pminsung12 deleted the feat/BSVR-256 branch September 9, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 CI/CD CI/CD 관련 ✨ Feature 기능 개발 🛠 Setting 개발 환경 세팅 size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants