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] feature: 개발용 토큰 발급 #152

Merged
merged 5 commits into from
Nov 29, 2023

Conversation

HKLeeeee
Copy link
Collaborator

@HKLeeeee HKLeeeee commented Nov 28, 2023

PR 설명

  • 개발용 토큰 발급을 위한 api입니다.

✅ 완료한 기능 명세

  • 개발용 api 설정
  • cors 설정
  • github action .env 파일 생성방식 변경

📸 스크린샷

고민과 해결과정

github action .env 파일 생성방식 변경

기존 env 파일 생성 방식으로는 새로운 환경변수 값이 생길 때 마다 yml, github secret 등록을 해주어야했고,
cd 파일이 너무너무 길어지는 불편함이 있었습니다.
env 파일 생성을 간편하게 하기 위해서 env 파일을 통채로 github secret에 등록하고 한 번에 읽어와 .env 파일을 만드는 방법으로 변경했습니다.

          script: |
            cd /root
            echo "${{secrets.API_ENV}}" > .env
            bash deploy.sh >> /dev/deploy.log 2>&1

script 가 간결해졌습니다!

로컬 환경에서 사용한 환경변수는 .env에 github secret에 넣어줄 배포환경의 환경변수는 .prod.env로 기록하며 별도로 관리하려고 합니다.

@HKLeeeee HKLeeeee added feature 기능 구현 BackEnd 백엔드 관련 labels Nov 28, 2023
@HKLeeeee HKLeeeee self-assigned this Nov 28, 2023
@HKLeeeee HKLeeeee requested a review from Gseungmin November 28, 2023 10:38
@HKLeeeee HKLeeeee changed the title feature: [BE] 개발용 토큰 발급 [BE] feature: 개발용 토큰 발급 Nov 28, 2023

echo "CLIENT_ID_GOOGLE=${{secrets.CLIENT_ID_GOOGLE}}" >> .env
echo "CLIENT_SECRET_GOOGLE=${{secrets.CLIENT_SECRET_GOOGLE}}" >> .env
echo "${{secrets.API_ENV}}" > .env
Copy link
Member

Choose a reason for hiding this comment

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

완전깔끔하네요!

@Gseungmin
Copy link
Member

LGTM

@Gseungmin Gseungmin merged commit 2fc80f3 into boostcampwm2023:dev Nov 29, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BackEnd 백엔드 관련 feature 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants