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] chore: 기존 환경변수를 유지하며 sh 실행하도록 변경 #888

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

nayonsoso
Copy link
Contributor

@nayonsoso nayonsoso commented Oct 21, 2024


🚀 어떤 기능을 구현했나요 ?

  • cd 파일에서 설정한 환경변수고 shell 스크립트로도 전달되게 했습니다.

🔥 어떻게 해결했나요 ?

  • 일반적으로 cd 파일의 환경변수는 그 하위에서 run 하는 shell script 에도 반영이 됩니다.
  • 하지만 우리가 sudo 로 스크립트를 실행하고 있어서 문제가 발생했습니다.
  • sudo는 보안상의 이유로 대부분의 환경 변수를 제거하고 새로운 환경에서 명령을 실행합니다.
  • 따라서 -E 옵션을 사용해서 현재 사용자의 환경 변수를 그대로 유지한 채 명령을 실행하도록 바꿔주었습니다.

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

Copy link
Contributor

@skylar1220 skylar1220 left a comment

Choose a reason for hiding this comment

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

정보)
sudo 모드일 때는 -E를 해야 스크립트에 변수 주입을 할 수 있다

@nayonsoso nayonsoso merged commit f0acaea into develop Oct 21, 2024
4 checks passed
@donghoony donghoony deleted the 885-zero-downtime-deployment-2 branch November 17, 2024 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BE] 무중단 배포를 적용한다.
4 participants