Skip to content

release/1.2.1#30

Open
Sean-mn wants to merge 8 commits intomainfrom
release/1.2.1
Open

release/1.2.1#30
Sean-mn wants to merge 8 commits intomainfrom
release/1.2.1

Conversation

@Sean-mn
Copy link
Copy Markdown
Contributor

@Sean-mn Sean-mn commented Mar 29, 2026

📚작업 내용

  • Azure Key Vault 의존성 완전 제거 (패키지 3개 삭제: Azure.Identity, Azure.Extensions.AspNetCore.Configuration.Secrets, Azure.Security.KeyVault.Secrets)
  • DB 연결 문자열을 환경변수(ConnectionStrings:Default) 기반으로 전환
  • DB 연결 문자열 빈 값 검증 강화 (IsNullOrWhiteSpace 적용)
  • Docker 빌드 경로 오류 수정 (PushAndPull/ServerPushAndPull/PushAndPull)
  • CI/CD 배포 시 Azure 인증 환경변수 3개를 DB_CONNECTION_STRING 시크릿 단일 환경변수로 교체
  • csproj 파일 BOM 제거

◀️참고 사항

기존에는 Azure Key Vault를 통해 DB 연결 문자열을 가져왔으나, 학교 서버(일반 VPS) 환경에서 Azure 서비스 의존성이 불필요하여 제거했습니다.

변경 전: Azure Key Vault + 서비스 프린시펄 자격증명(AZURE_CLIENT_ID/SECRET/TENANT_ID)을 Docker 환경변수로 주입
변경 후: GitHub Secret DB_CONNECTION_STRING을 Docker 환경변수 ConnectionStrings__Default로 직접 주입

✅체크리스트

[ ]안에 x를 작성하면 체크박스를 체크할 수 있습니다.

  • 현재 의도하고자 하는 기능이 정상적으로 작동하나요?
  • 변경한 기능이 다른 기능을 깨뜨리지 않나요?

추후 필요한 체크리스트는 업데이트 될 예정입니다.

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 transitions the application from Azure Key Vault-based connection string management to standard configuration strings. It removes Azure-related dependencies and initialization code from the project file and Program.cs, updates the appsettings files, and refactors the database configuration logic. A review comment suggests changing the new English exception message in DatabaseConfig.cs to Korean to maintain consistency with the existing codebase.

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