fix: 配置管理规范化 - 修复 .env.production.example 变量名不一致 #124

2026-06-30T15:51:29+08:00

xiaoxia commented

2026-06-30 15:51:29 +08:00

配置管理规范化修复

问题

对照代码中所有 os.getenv() 和 Settings 类，发现 .env.production.example 存在多处不一致。

修复内容

BASE_URL → APP_BASE_URL
CORS_ORIGINS → CORS_ORIGINS_RAW（逗号分隔格式）
新增 APP_ENV、OSS 配置、Celery 配置等 18 个缺失变量
SENTRY_DSN 注释掉（代码中未实际使用）

## 配置管理规范化修复 ### 问题对照代码中所有 os.getenv() 和 Settings 类，发现 .env.production.example 存在多处不一致。 ### 修复内容 - BASE_URL → APP_BASE_URL - CORS_ORIGINS → CORS_ORIGINS_RAW（逗号分隔格式） - 新增 APP_ENV、OSS 配置、Celery 配置等 18 个缺失变量 - SENTRY_DSN 注释掉（代码中未实际使用）

xiaoxia added 1 commit 2026-06-30 15:51:29 +08:00

fix: 配置管理规范化 - 修复 .env.production.example 变量名不一致问题

CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Has been cancelled

Details

CI/CD Pipeline / Frontend Lint (pull_request) Has been cancelled

Details

e0a24f636d

主要修复：
- BASE_URL → APP_BASE_URL（与代码 Settings.APP_BASE_URL 一致）
- CORS_ORIGINS (JSON数组) → CORS_ORIGINS_RAW (逗号分隔，与代码一致)
- 新增 APP_ENV（validate_release_env.py 标记为必需）
- 新增 OSS 配置：OSS_ENDPOINT, OSS_ACCESS_KEY_ID, OSS_ACCESS_KEY_SECRET, OSS_BUCKET_NAME
- 新增 OSS_DIRECT_UPLOAD_MAX_MB, OSS_DIRECT_UPLOAD_EXPIRE_SECONDS
- 新增 ENABLE_REDIS_SESSIONS, ENABLE_EMAIL_DELIVERY, SMTP_USE_TLS, SMTP_FROM_NAME
- 新增 GENERATED_FILES_DIR, GENERATED_FILES_URL_PREFIX, PUBLIC_API_BASE_URL
- 新增 CELERY_BROKER_URL, CELERY_RESULT_BACKEND
- SMTP 占位符格式改为 CHANGE_ME_ 前缀（通过验证脚本检查）
- SENTRY_DSN 注释掉（代码中未实际使用）

xiaoxia commented

2026-06-30 17:10:15 +08:00

🔍 代码审查 — PR #124

结论：✅ 审查通过，可以合并。

变更分析

单文件变更：.env.production.example（+36/-12）

改进项：

改动	评价
添加分区标题	✅ 结构清晰，便于运维定位
`BASE_URL` → `APP_BASE_URL`	✅ 与 config.py 字段名对齐
`CORS_ORIGINS` → `CORS_ORIGINS_RAW`	✅ 与 config.py 实际字段名一致，逗号分隔格式正确
SMTP 占位符改为 `CHANGE_ME_*`	✅ 比 `your-email@gmail.com` 更清晰，不会被误用
新增 OSS 配置块（6个变量）	✅ 与 config.py 字段完全匹配，默认值合理
新增 Celery 配置块	✅ `CELERY_BROKER_URL` / `CELERY_RESULT_BACKEND` 与 config.py 一致
新增邮件开关 `ENABLE_EMAIL_DELIVERY` / `SMTP_USE_TLS` / `SMTP_FROM_NAME`	✅ 与 config.py 字段匹配
新增 `GENERATED_FILES_DIR` / `GENERATED_FILES_URL_PREFIX`	✅ 与 main.py 中 `os.getenv()` 调用一致
`SENTRY_DSN` 改为注释	✅ 可选配置，注释掉合理

与 config.py 一致性验证

逐一比对全部 36 个变量，与 config.py Settings 类字段及 main.py os.getenv() 调用完全一致，无遗漏、无拼写错误。

P3 观察（不阻塞合并）

PUBLIC_API_BASE_URL 在后端代码中未发现引用，可能是前端或部署脚本使用，建议确认
APP_ENV=production 被 get_settings() 通过 os.getenv() 读取来选择加载哪个 .env.* 文件，属于运行时元配置，放在 .env.production.example 中逻辑上正确

审查结论：✅ 通过。配置模板规范化到位，变量命名与代码完全一致。建议合并。

## 🔍 代码审查 — PR #124 **结论：✅ 审查通过，可以合并。** --- ### 变更分析单文件变更：`.env.production.example`（+36/-12） **改进项：** | 改动 | 评价 | |------|------| | 添加分区标题 | ✅ 结构清晰，便于运维定位 | | `BASE_URL` → `APP_BASE_URL` | ✅ 与 config.py 字段名对齐 | | `CORS_ORIGINS` → `CORS_ORIGINS_RAW` | ✅ 与 config.py 实际字段名一致，逗号分隔格式正确 | | SMTP 占位符改为 `CHANGE_ME_*` | ✅ 比 `your-email@gmail.com` 更清晰，不会被误用 | | 新增 OSS 配置块（6个变量） | ✅ 与 config.py 字段完全匹配，默认值合理 | | 新增 Celery 配置块 | ✅ `CELERY_BROKER_URL` / `CELERY_RESULT_BACKEND` 与 config.py 一致 | | 新增邮件开关 `ENABLE_EMAIL_DELIVERY` / `SMTP_USE_TLS` / `SMTP_FROM_NAME` | ✅ 与 config.py 字段匹配 | | 新增 `GENERATED_FILES_DIR` / `GENERATED_FILES_URL_PREFIX` | ✅ 与 main.py 中 `os.getenv()` 调用一致 | | `SENTRY_DSN` 改为注释 | ✅ 可选配置，注释掉合理 | ### 与 config.py 一致性验证逐一比对全部 36 个变量，与 config.py Settings 类字段及 main.py `os.getenv()` 调用**完全一致**，无遗漏、无拼写错误。 ### P3 观察（不阻塞合并） 1. `PUBLIC_API_BASE_URL` 在后端代码中未发现引用，可能是前端或部署脚本使用，建议确认 2. `APP_ENV=production` 被 `get_settings()` 通过 `os.getenv()` 读取来选择加载哪个 `.env.*` 文件，属于运行时元配置，放在 `.env.production.example` 中逻辑上正确 --- **审查结论：✅ 通过。配置模板规范化到位，变量命名与代码完全一致。建议合并。**

xiaoxia merged commit 51017066e0 into develop

2026-06-30 17:17:01 +08:00

xiaoxia referenced this issue from a commit

2026-06-30 17:17:02 +08:00

fix: 配置管理规范化 - 修复 .env.production.example 变量名不一致 (#124)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#124