chore(ci): 端口与PG配置常量集中管理，清理硬编码 #811

Merged

xiaoxia merged 1 commits from ci/port-env-cleanup into develop 2026-07-24 11:43:05 +08:00

Conversation 2 Commits 1 Files Changed 5

+62 -23

xiaoxia commented 2026-07-24 10:52:31 +08:00

Owner

Copy Link

Copy Source

变更内容

新增

• scripts/ci/ci_env.sh — CI共享常量文件，集中管理端口/用户/密码/DB名等配置

修改

• scripts/ci/validate_migration.sh — source ci_env.sh，移除SHARED_PG_PORT/USER/PASSWORD硬编码
• scripts/ci/run_validate.sh — 同上
• scripts/ci/run_integration_tests.sh — 同上
• .gitea/workflows/ci-pipeline.yml — workflow级env变量统一管理，两处DATABASE_URL从硬编码改为变量引用

清理清单

• 3个脚本中的 SHARED_PG_PORT="5433" → ${CI_SHARED_PG_PORT}
• 3个脚本中的 SHARED_PG_USER="postgres" → ${CI_SHARED_PG_USER}
• 3个脚本中的 SHARED_PG_PASSWORD="ci_pg_2026!" → ${CI_SHARED_PG_PASSWORD}
• yml中2处 host.docker.internal:5432 硬编码 → ${{ env.CI_PG_HOST }}:${{ env.CI_PG_PORT }}

好处

• 配置统一管理，修改一处全局生效
• 支持通过环境变量覆盖默认值（如Runner配置变化时无需改代码）
• 消除重复硬编码，降低维护成本

## 变更内容 ### 新增 - `scripts/ci/ci_env.sh` — CI共享常量文件，集中管理端口/用户/密码/DB名等配置 ### 修改 - `scripts/ci/validate_migration.sh` — source ci_env.sh，移除SHARED_PG_PORT/USER/PASSWORD硬编码 - `scripts/ci/run_validate.sh` — 同上 - `scripts/ci/run_integration_tests.sh` — 同上 - `.gitea/workflows/ci-pipeline.yml` — workflow级env变量统一管理，两处DATABASE_URL从硬编码改为变量引用 ### 清理清单 - 3个脚本中的 `SHARED_PG_PORT="5433"` → `${CI_SHARED_PG_PORT}` - 3个脚本中的 `SHARED_PG_USER="postgres"` → `${CI_SHARED_PG_USER}` - 3个脚本中的 `SHARED_PG_PASSWORD="ci_pg_2026!"` → `${CI_SHARED_PG_PASSWORD}` - yml中2处 `host.docker.internal:5432` 硬编码 → `${{ env.CI_PG_HOST }}:${{ env.CI_PG_PORT }}` ### 好处 - 配置统一管理，修改一处全局生效 - 支持通过环境变量覆盖默认值（如Runner配置变化时无需改代码） - 消除重复硬编码，降低维护成本

xiaoxia added 1 commit 2026-07-24 10:52:31 +08:00

chore(ci): 端口与PG配置常量集中管理，清理硬编码 ... 764ac311ee

- 新增 scripts/ci/ci_env.sh 统一管理CI共享常量（端口/用户/密码/DB名）
- 3个CI脚本（validate_migration/run_validate/run_integration_tests）
  source ci_env.sh，移除SHARED_PG_PORT/USER/PASSWORD等硬编码
- ci-pipeline.yml新增workflow级env变量，两处DATABASE_URL从硬编码改为变量引用
- 支持通过环境变量覆盖默认值，便于后续配置调整

auto-approve-bot commented 2026-07-24 11:18:22 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #811

⚠️ 问题（2个需要修改）

1. scripts/ci/run_integration_tests.sh 第228行：临时容器使用了错误的数据库凭据

   • 问题描述：在 else 分支（启动临时 PostgreSQL 容器）中，DATABASE_URL 使用了 ${CI_SHARED_PG_USER} 和 ${CI_SHARED_PG_PASSWORD}。
   • 后果：CI_SHARED_PG_PASSWORD 是用于连接“常驻共享 PG 实例”的密码（ci_pg_2026!）。新启动的 postgres:16 临时容器默认并未设置此密码（除非 docker run 命令中未显示的 diff 部分传递了 -e POSTGRES_PASSWORD，否则默认为空或 trust）。这将导致在 CI_USE_SHARED_PG=false 时，数据库连接认证失败，CI 任务崩溃。

2. scripts/ci/run_validate.sh 第463行：临时容器使用了错误的数据库凭据

   • 问题描述：同上，在 else 分支中构建 DATABASE_URL 时使用了共享实例的密码变量 ${CI_SHARED_PG_PASSWORD}。
   • 后果：导致临时数据库连接认证失败。

3. .gitea/workflows/ci-pipeline.yml 第33行：敏感信息硬编码

   • 问题描述：CI 配置文件和 ci_env.sh 中硬编码了数据库密码 ci_pg_2026!。
   • 后果：虽然这是 CI 配置，但将具有特定含义的密码明文提交到代码库存在安全风险。如果该密码用于其他环境或包含敏感信息，应迁移到 Gitea Secrets 中管理。

💡 建议（1个可选）

1. scripts/ci/ci_env.sh：增加临时数据库的独立配置项
   • 具体内容：当前 ci_env.sh 仅定义了 CI_SHARED_PG_* 变量，但在脚本中被误用于临时容器。建议增加 CI_LOCAL_PG_USER 和 CI_LOCAL_PG_PASSWORD（默认为 postgres），并在脚本的 else 分支中使用这些独立变量，以明确区分“共享实例”和“临时实例”的配置，避免逻辑混淆。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 03:18:22 | 模型:}

## 代码审查结果 - PR #811 ### ⚠️ 问题（2个需要修改） 1. **scripts/ci/run_integration_tests.sh 第228行**：**临时容器使用了错误的数据库凭据** - **问题描述**：在 `else` 分支（启动临时 PostgreSQL 容器）中，`DATABASE_URL` 使用了 `${CI_SHARED_PG_USER}` 和 `${CI_SHARED_PG_PASSWORD}`。 - **后果**：`CI_SHARED_PG_PASSWORD` 是用于连接“常驻共享 PG 实例”的密码（`ci_pg_2026!`）。新启动的 `postgres:16` 临时容器默认并未设置此密码（除非 `docker run` 命令中未显示的 diff 部分传递了 `-e POSTGRES_PASSWORD`，否则默认为空或 trust）。这将导致在 `CI_USE_SHARED_PG=false` 时，数据库连接认证失败，CI 任务崩溃。 2. **scripts/ci/run_validate.sh 第463行**：**临时容器使用了错误的数据库凭据** - **问题描述**：同上，在 `else` 分支中构建 `DATABASE_URL` 时使用了共享实例的密码变量 `${CI_SHARED_PG_PASSWORD}`。 - **后果**：导致临时数据库连接认证失败。 3. **.gitea/workflows/ci-pipeline.yml 第33行**：**敏感信息硬编码** - **问题描述**：CI 配置文件和 `ci_env.sh` 中硬编码了数据库密码 `ci_pg_2026!`。 - **后果**：虽然这是 CI 配置，但将具有特定含义的密码明文提交到代码库存在安全风险。如果该密码用于其他环境或包含敏感信息，应迁移到 Gitea Secrets 中管理。 ### 💡 建议（1个可选） 1. **scripts/ci/ci_env.sh**：**增加临时数据库的独立配置项** - **具体内容**：当前 `ci_env.sh` 仅定义了 `CI_SHARED_PG_*` 变量，但在脚本中被误用于临时容器。建议增加 `CI_LOCAL_PG_USER` 和 `CI_LOCAL_PG_PASSWORD`（默认为 `postgres`），并在脚本的 `else` 分支中使用这些独立变量，以明确区分“共享实例”和“临时实例”的配置，避免逻辑混淆。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 03:18:22 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 94afec2b9c into develop 2026-07-24 11:43:05 +08:00

gitea-actions bot commented 2026-07-24 12:06:00 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #811 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #811 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#811