Skip to content

refactor: NCP ObjectStorage 설정#150

Merged
ekdan38 merged 1 commit into
mainfrom
feature/ncp-storage
Mar 31, 2026
Merged

refactor: NCP ObjectStorage 설정#150
ekdan38 merged 1 commit into
mainfrom
feature/ncp-storage

Conversation

@ekdan38
Copy link
Copy Markdown
Collaborator

@ekdan38 ekdan38 commented Mar 31, 2026

🔥ISSUE

NCP ObjectStorage 설정

@ekdan38 ekdan38 merged commit 04bd908 into main Mar 31, 2026
1 check passed
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds environment variables for NCP (Naver Cloud Platform) access and storage to the production Docker Compose configuration. The feedback suggests using mandatory environment variable syntax to ensure these keys are provided at runtime and recommends adopting more secure secret management practices for sensitive credentials.

Comment thread docker-compose.prod.yml
Comment on lines +82 to +84
NCP_ACCESS_KEY: ${NCP_ACCESS_KEY}
NCP_SECRET_KEY: ${NCP_SECRET_KEY}
NCP_BUCKET_NAME: ${NCP_BUCKET_NAME}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

운영 환경에서는 NCP 액세스 키와 버킷 이름과 같은 필수 설정이 누락되지 않도록 보장하는 것이 중요합니다. 리포지토리 규칙에 따라 YAML 설정 파일에서 환경 변수를 사용해야 하며, ${VARIABLE:?error} 구문을 활용하면 환경 변수가 설정되지 않았을 때 배포 단계에서 에러를 발생시켜 잘못된 설정으로 애플리케이션이 실행되는 것을 방지할 수 있습니다.

      NCP_ACCESS_KEY: ${NCP_ACCESS_KEY:?NCP_ACCESS_KEY must be set}
      NCP_SECRET_KEY: ${NCP_SECRET_KEY:?NCP_SECRET_KEY must be set}
      NCP_BUCKET_NAME: ${NCP_BUCKET_NAME:?NCP_BUCKET_NAME must be set}
References
  1. 환경별 설정 시 YAML 파일에서 주석 처리 대신 기본값이 포함된 환경 변수를 사용해야 합니다.

Comment thread docker-compose.prod.yml
COOKIE_SAME_SITE: ${COOKIE_SAME_SITE}
COOKIE_DOMAIN: ${COOKIE_DOMAIN}
NCP_ACCESS_KEY: ${NCP_ACCESS_KEY}
NCP_SECRET_KEY: ${NCP_SECRET_KEY}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

security-medium medium

NCP_SECRET_KEY와 같은 보안 민감 정보를 환경 변수로 직접 노출하는 것은 보안상 위험할 수 있습니다. 프로세스 목록이나 로그 등을 통해 유출될 가능성이 있으므로, 향후 Docker Secrets나 외부 Secret 관리 솔루션 도입을 고려해 보시기 바랍니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant