Skip to content

Conversation

ezcolin2
Copy link
Collaborator

@ezcolin2 ezcolin2 commented Jan 18, 2025

🔖 연관된 이슈

📂 작업 내용

  • docker-compose 파일 environment 제거
  • docker-compose 파일 .env bind mount 제거
  • env_file로 .env 파일 위치 알려주는 방식으로 변경
  • JwtAuthGuard 주입되지 않아서 AuthController test 실패하던 이슈 해결
  • reviewer에서 현준님 제거

저번 피어세션 시간에 보안을 위해 .env 파일을 bind mount했다고 발표를 하니 한 분께서 docker-compose에서 제공해주는 env_file을 사용하면 bind mount 없이도 쉽게 환경 변수를 주입할 수 있다고 하셔서 적용해보았습니다.

bind mount 자체를 없앨 수 있어서 좋은 것 같습니다.

그리고 AuthController test에 JwtAuthGuard 주입 추가했고 현준님 자동 리뷰어로 등록되는 현상 있어서 현준님 리뷰어 그룹에서 제거했습니다.

📑 참고 자료

https://docs.docker.com/compose/how-tos/environment-variables/set-environment-variables/

@ezcolin2 ezcolin2 linked an issue Jan 18, 2025 that may be closed by this pull request
3 tasks
@ezcolin2 ezcolin2 marked this pull request as draft January 18, 2025 08:13
@ezcolin2 ezcolin2 added 🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 ♻️ Refactor 기능 추가 외 코드 변경, 코드 품질과 관련된 경우 labels Jan 19, 2025
@ezcolin2 ezcolin2 marked this pull request as ready for review January 19, 2025 00:48
@ezcolin2 ezcolin2 changed the title Refactor be #45 docker 배포 환경 개선 Jan 19, 2025
@ezcolin2 ezcolin2 marked this pull request as draft January 19, 2025 00:49
@ezcolin2 ezcolin2 removed the request for review from Tolerblanc January 19, 2025 00:51
@github-actions github-actions bot requested a review from Tolerblanc January 19, 2025 00:51
@ezcolin2 ezcolin2 removed the request for review from Tolerblanc January 19, 2025 00:57
@ezcolin2 ezcolin2 changed the title docker 배포 환경 개선 docker .env bind mount 제거 Jan 19, 2025
@ezcolin2 ezcolin2 force-pushed the refactor-be-#45 branch 3 times, most recently from b92783b to 250deaf Compare January 19, 2025 14:55
@ezcolin2 ezcolin2 marked this pull request as ready for review January 19, 2025 14:57
@github-actions github-actions bot merged commit a72e42d into develop Jan 20, 2025
14 checks passed
@github-actions github-actions bot deleted the refactor-be-#45 branch January 20, 2025 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 ♻️ Refactor 기능 추가 외 코드 변경, 코드 품질과 관련된 경우

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docker-compose yml 파일 bind mount 제거

2 participants