Remove the committed connection string default and load secrets from Key Vault
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
Bắt đầu với LearnStack/appsettings.json:2-4 và LearnStack/Program.cs:32-33 để theo dõi cách chuỗi kết nối hiện đang được tải và xác thực. Sau đó xem xét đường dẫn cấu hình production cho Key Vault hoặc managed identity. Hoàn tất khi không còn chuỗi kết nối nào trong source control, production sử dụng managed identity và cấu hình không đầy đủ tạo ra lỗi khởi động rõ ràng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Tier 2 — configuration
Problem
appsettings.json ships a LocalDB connection string as the default. In production this is dead weight at best; at worst, if the environment variable is missing, the app starts and tries to reach a database that does not exist rather than failing loudly with a clear message.
Evidence
LearnStack/appsettings.json:2-4LearnStack/Program.cs:32-33— throws only when the value is entirely absent
Proposed fix
- Remove the default from
appsettings.json; keep it inappsettings.Development.jsonor user secrets only. - Source the production connection string from Key Vault via managed identity, or use a passwordless Azure SQL connection with managed identity.
- Fail fast at startup with an actionable message when configuration is incomplete.
Acceptance criteria
- No connection string in source control
- Production connects with managed identity
- Missing configuration produces a clear startup error
- Ngôn ngữ chính
- HTML
- Star
- 10
- Fork
- 0
- Merge trung bình
- 2 giờ 35 phút
- Pull request đã merge (30 ngày)
- 21
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của kasuken/LearnStack
-
enhancement
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 91/100
kasuken/LearnStack#59 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
kasuken/LearnStack#72 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Feature: admin dashboardĐang mởenhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
kasuken/LearnStack#71 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
kasuken/LearnStack#70 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Feature: public profile pagesĐang mởenhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 45/100
kasuken/LearnStack#69 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của kasuken/LearnStack
Issue tương tự
-
cloudflare: root entry eagerly imports `wrangler`, so `d1`/`r2`/`kvCache` cannot load without a Wrangler installCó thể đã có người làm @emdashbot đã nhận hôm nay. Đang mởarea/cloudflare bot:bug bot:needs-attention
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
emdash-cms/emdash#4232 · 4 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
auto-shift/autoshiftv2#282 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
oblien/openship#1310 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
voidzero-dev/void#463 ·
Maintainer thường phản hồi trong vòng 1 ngày