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

AWS ECS 환경 변수 관리 방식으로 Dockerfile 수정 #62

Merged

Conversation

yuminn-k
Copy link
Contributor

🔍 이 PR로 해결하고자 하는 문제는 무엇인가요?

  • Dockerfile에서 민감한 데이터를 ARG와 ENV로 설정하는 보안 문제를 해결하고, AWS ECS의 task 정의를 통해 환경 변수를 관리하도록 변경했습니다.

✨ 이 PR로 주요하게 바뀐 점은 무엇인가요?

  • Dockerfile에서 ARG와 ENV 지시어를 제거하고, 필요한 파일만 복사하도록 수정했습니다.
  • 환경 변수는 AWS ECS의 task 정의에서 관리하도록 변경했습니다.

🔖 주요 변경 사항 외에 추가로 변경된 부분이 있나요?

  • 없음

🙏🏻 리뷰어가 특히 봐주었으면 하는 부분은 무엇인가요?

  • Dockerfile 수정 사항이 최적화되었는지 확인 부탁드립니다.

🩺 이 PR로 테스트나 검증이 필요한 부분이 있나요?

  • Docker 이미지 빌드 및 AWS ECS 배포 테스트가 필요합니다.

📚 관련된 Issue나 Notion, 문서

🖥 작동하는 모습

스크린샷이나 녹화된 비디오, 또는 gif를 추가해서, 리뷰어가 변경점을 이해하는 데 도움이 되도록 해주세요.

📌 PR을 작성할 때의 주의사항

  • 리뷰어는 코드 리뷰 시 좋은 코드의 방향성을 제시하지만, 코드 수정을 강요하지 않습니다.
  • 리뷰어는 좋은 코드를 발견하면 칭찬과 격려를 아끼지 않습니다.
  • 리뷰는 특별한 경우가 아니라면, 리뷰어로 지정된 시점부터 3일 이내에 완료해주세요.
  • 댓글 작성 시 Prefix에 P1, P2, P3을 작성해주시면, 담당자가 댓글에 대해 더 명확히 대응할 수 있습니다.
    • P1 : 반드시 반영해주세요 (Request Changes) - 문제가 발생하거나, 취약점이 발견된 경우 등.
    • P2 : 반영을 적극적으로 검토해주세요 (댓글).
    • P3 : 이런 방법도 있을 것 같다는 사소한 의견입니다 (Chore).

…finition

- Removed ARG and ENV instructions for sensitive data
- Updated Dockerfile to copy necessary files only
- Ensured build and runtime stages are optimized

Related to ryokushaka#37
…date' into ci/37-add-ecs-task-definition-update
@yuminn-k yuminn-k added ⏰ P0 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🐞 BugFix Something isn't working 🆗 safe CI workflow 동작 관련 🌏 Deploy 배포 관련 labels Oct 14, 2024
@yuminn-k yuminn-k self-assigned this Oct 14, 2024
…finition

- Removed ARG and ENV instructions for sensitive data
- Updated Dockerfile to copy necessary files only
- Ensured build and runtime stages are optimized

Related to ryokushaka#37
@yuminn-k yuminn-k merged commit 2a22bd1 into ryokushaka:main Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix Something isn't working 🌏 Deploy 배포 관련 ⏰ P0 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🆗 safe CI workflow 동작 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant