fix(version): /health 返回真实 commit SHA，不再是 unknown #2058

Merged

auto-approve-bot merged 1 commits from fix/app-version-inject into develop 2026-09-27 01:13:35 +08:00

Conversation 2 Commits 1 Files Changed 2

+3 -5

xiaoxia commented 2026-09-27 00:59:27 +08:00

Owner

Copy Link

Copy Source

问题

staging 部署后 GET /health 返回 version="unknown"，Docker 构建时注入的 commit SHA 被 compose 默认值覆盖。

根因

infra/docker/compose.yml 里硬编码 APP_VERSION: ${APP_VERSION:-unknown}，compose 启动时宿主机未设 APP_VERSION 就注入 unknown，覆盖 Dockerfile 里的 ENV APP_VERSION=。

修复

1. 删除 compose.yml 两处 APP_VERSION 硬编码，让镜像 ENV APP_VERSION 自然生效；需要 override 时仍可通过 -e 传入。
2. ci-pipeline.yml 去掉 EXTRA_BUILD_ARGS 中 APP_VERSION 值的多余转义引号。

## 问题 staging 部署后 GET /health 返回 version="unknown"，Docker 构建时注入的 commit SHA 被 compose 默认值覆盖。 ## 根因 infra/docker/compose.yml 里硬编码 `APP_VERSION: ${APP_VERSION:-unknown}`，compose 启动时宿主机未设 APP_VERSION 就注入 unknown，覆盖 Dockerfile 里的 ENV APP_VERSION=<sha>。 ## 修复 1. 删除 compose.yml 两处 APP_VERSION 硬编码，让镜像 ENV APP_VERSION 自然生效；需要 override 时仍可通过 -e 传入。 2. ci-pipeline.yml 去掉 EXTRA_BUILD_ARGS 中 APP_VERSION 值的多余转义引号。

xiaoxia added 1 commit 2026-09-27 00:59:27 +08:00

fix(version): 让镜像构建时注入的 APP_VERSION (commit SHA) 正确生效 ... 0958c3db1a

- infra/docker/compose.yml: 移除 environment 里 APP_VERSION=${APP_VERSION:-unknown} 硬编码默认值，
  避免 compose 启动时用 "unknown" 覆盖 Dockerfile ARG/ENV 注入的 commit SHA。
  未显式传 -e APP_VERSION 时，容器使用镜像 ENV（即构建时 --build-arg APP_VERSION=<sha> 的值）；
  需要 override 时仍可通过 docker compose -e 或宿主机环境变量传入。
- .gitea/workflows/ci-pipeline.yml: 去掉 EXTRA_BUILD_ARGS 中 APP_VERSION 值的多余转义引号，
  避免 build-arg 把字面双引号注入版本号（之前 version 显示为 "\"abc123\"" 而非 "abc123"）。

gitea-actions bot commented 2026-09-27 01:01:29 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#2058
预览链接	https://pr-2058.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #2058 | | 预览链接 | [https://pr-2058.preview.xiaoxiajianji.com](https://pr-2058.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot merged commit 1463cf0834 into develop 2026-09-27 01:13:35 +08:00

auto-approve-bot deleted branch fix/app-version-inject 2026-09-27 01:13:37 +08:00

gitea-actions bot commented 2026-09-27 01:17:54 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot frontend-dev xiaoxia

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#2058