Skip to content

fix: remove default credentials and patch runtime vulnerabilities - #333

Merged
DavidHLP merged 7 commits into
mainfrom
fix/gitguardian-env-template
Nov 6, 2026
Merged

DavidHLP merged 7 commits into
mainfrom
fix/gitguardian-env-template

Conversation

@DavidHLP

@DavidHLP DavidHLP commented Oct 30, 2026 •

Copy link
Copy Markdown
Owner

修复范围

  • 移除已跟踪环境模板中的固定凭据;初始化随机生成管理员密码并同步浏览器验收输入,保留 Redis/Compose 必填校验及 CI 专用合成配置。
  • 补齐初始化随机性、密码一致性、0600 文件权限、输出不泄漏、保留已有配置及强制重建回归。
  • 管理员 bootstrap 启用且缺少/空密码时 fail closed,不再回退固定密码;显式禁用、prepare-only、未启用 Auth 的路径保持跳过。
  • 统一 Vue I18n 11.4.16、follow-redirects 1.16.2,通过 Stylus 0.59.0 移除旧 URI decoder 链;六个后端服务使用 Nacos 2.5.1 pure SDK 和同版本 API/common,排除默认 shaded client,由 Maven 管理 Guava 33.7.2-jre。
  • 增加固定白名单诊断、日志覆盖信息、最终 readiness/dependency 枚举;固定 loopback 探测限制响应大小,不跟随重定向,整体诊断采用 25 秒超时及 2 秒强制终止上限。保留原就绪门禁、失败退出和清理流程,不发布原始日志、响应或凭据。

未降低现有审计、扫描或测试门槛。

当前代码与验证

分支:fix/gitguardian-env-template
当前 head:86f334b6bd3dda2bf1b0f2e6b92ede7a65efee6d

后续提交:

  • 19bec2e44bf6dbeebc2055dc5f33d5587646ad3b:初始化文档与契约回归。
  • 64ed01b23e65ae4698e44955f6bf86aab94775b6:受限诊断、bootstrap 缺密码失败退出及回归。
  • 86f334b6bd3dda2bf1b0f2e6b92ede7a65efee6d:最终探测及整体超时兜底。

CI run 41290201823 已完成并成功,包括前端审计/覆盖率/构建、后端构建与三组测试、迁移/兼容性、容器构建扫描、Agent、Secret Scan 与真实干净安装/业务生命周期验收。GitGuardian PR 检查通过。
ci-ok 124803367325 的“Verify every selected gate succeeded”已核验成功;先前列表中未返回聚合记录的暂时证据缺口已澄清,不能再称 ci-ok 缺失。

真实验收测试合并提交为 e41636b0e97916d8d9391a9da2974a37b1777d96,对应 base 4a2ce8e3ce51856a65be66d92d5fbb4cbcfd187f 与上述 head;14 项命名检查通过,覆盖浏览器登录、提交、真实 Judge Accepted/Wrong Answer、题目编辑读回、重判及比赛排名。
新增逻辑的 31 项针对性回归(12 诊断报告、9 bootstrap、10 readiness)通过,包含敏感内容不回显、缺失密码、挂住子进程终止及清理。针对性测试不替代完整远程 CI。

尚未消除的风险与限制

  1. Nacos protobuf 旧生成代码风险尚未消除。 Metadata/Payload 警告仍存在。Guava 更新、消息往返和 loopback 测试不证明资源放大风险消失,也不证明公网可利用;仍需兼容生成代码修复及受限异常输入、真实注册/鉴权/订阅/重连验证。
  2. 生产依赖审计仍有 1 MODERATE。 现有 high 门槛通过,但可读摘要尚不足以确定 advisory/依赖路径;不猜测具体包,不将多个应用重复摘要计为不同漏洞。
  3. 此前间歇启动失败未证明根治。 新诊断与本轮成功真实验收不等于确定根因已消除。
  4. 历史凭据事件未关闭。 未撤销/轮换真实凭据、未重写 Git 历史、未关闭/忽略 GitGuardian 历史告警。PR 检查通过不代表所有账户事件解决。
  5. 旧云端记录不作为证据。 84e24dad… 原工作区与日志不可访问;本说明仅引用当前可核验远程提交与 CI,不重建旧记录。

当前 PR 状态

已 Ready,尚未合并或部署。2026-11-01 10:58 UTC 读回为 mergeable=true、mergeable_state=blocked;具体阻塞原因未确证,且不是 ci-ok 缺失。不降低保护、不伪造检查,不进行生产发布。

DavidHLP and others added 3 commits October 29, 2026 22:37
Signed-off-by: DavidHLP <lysf15520112973@163.com>
Signed-off-by: DavidHLP <lysf15520112973@163.com>
Signed-off-by: DavidHLP <lysf15520112973@163.com>
@DavidHLP DavidHLP changed the title fix: remove committed default environment credentials fix: remove default credentials and patch runtime vulnerabilities Nov 1, 2026
@DavidHLP
DavidHLP marked this pull request as ready for review November 1, 2026 05:53
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-06T10:11:28.613515Z 15f0c7b New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@DavidHLP
DavidHLP merged commit 0a771f7 into main Nov 6, 2026
31 checks passed
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