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

.env 파일 전송 #136

Merged
merged 8 commits into from
Nov 13, 2024
Merged

.env 파일 전송 #136

merged 8 commits into from
Nov 13, 2024

Conversation

ezcolin2
Copy link
Collaborator

@ezcolin2 ezcolin2 commented Nov 13, 2024

🔖 연관된 이슈

📂 작업 내용

.env 파일은 .gitignore에 등록되어 깃허브에 올라가지 않습니다.

.env 파일을 github action secrets에 등록하고 push가 발생했을 때 .env 파일을 원격 서버로 전송합니다.

그 후 deploy.sh도 원격 서버로 보내서 쉘 스크립트 파일을 실행합니다.

deploy.sh는 github에서 최신 변경 사항을 가져온 다음 3000번 포트에 서버를 띄웁니다.

  • .

📢 리뷰 요구사항 (선택)

지금 뭔가 CD를 야매로 한 것 같아서 걱정이 되네요.

git에서 최신 정보를 pull 해 올 경우 .env 파일이 없어서 데이터베이스 연동이 안됩니다.

이를 해결하기 위해 github action secrets에 .env를 등록한 다음 이 .env 파일을 원격 서버로 보내도록 하고 있습니다.

.env 파일을 노출시키지 않으면서 CD를 구축하기 위해 선택한 방법인데 혹시 다른 좋은 방법 있으시다면 공유해주시면 감사하겠습니다!!

@ezcolin2 ezcolin2 added 🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 🐛 Bug 무언가 정상적으로 동작하지 않는 경우 labels Nov 13, 2024
@ezcolin2 ezcolin2 requested a review from Tolerblanc November 13, 2024 11:53
Copy link
Collaborator

@summersummerwhy summersummerwhy left a comment

Choose a reason for hiding this comment

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

연관된 이슈번호 잘못 쓰신것 같아서 수정하고 머지할게요!! 수고하셨습니다ㅠㅜ!!!

@github-actions github-actions bot merged commit e675719 into develop Nov 13, 2024
20 of 25 checks passed
@github-actions github-actions bot deleted the feature-be-#110 branch November 13, 2024 12:36
@Tolerblanc
Copy link
Member

그래도 최선의 방식을 선택하신 것 같은데요? 저희만의 private 레포를 둘 수 없으니까 지금 이상의 방식이 잘 떠오르지 않네요. 수고 많으셨습니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 🐛 Bug 무언가 정상적으로 동작하지 않는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants