`api_key=volcengine.api_key` 这类**依赖注入传参**被 KEY-ASSIGN 判为疑似凭证。 这是规则的真缺口而非代码问题:行里没有任何秘密,而做 DI 的代码到处都是这种 写法——绕过一次就得在每个调用点绕过,那等于把规则废掉。故改规则。 新增 ALLOW_REFERENCE,刻意收得很紧以免开后门: - 值必须是**未加引号**的小写起首标识符引用(真密钥要么带引号,要么不是标识符形状) - 只允许**恰好一个点且随即行尾** - 连字符不在字符集内 对抗性夹具验证(三真三假): - 拦下 `api_key = "sk-T3Blbk…"`(OpenAI 形态) - 拦下 `secret_key: "AKID…"`(腾讯云形态) - 拦下 `auth_token=eyJ….eyJ….Sfl…`(**三段式 JWT——两个点,故不落进放行**) - 放行 `api_key=lowercase.reference` / `api_key = settings.api_key` / `api_token: config.authToken` 原 5 形态泄漏夹具复验仍 8 条命中,一条不少。 另更正口径:脚本头与 git-workflow.md 仍写「三仓同构」,实际已是四仓—— patbond-ai 是唯一持有 AI 服务商凭证的仓,漏同步它风险最大。四仓 md5 已一致 (4945940093b4b8413b7d3eec8bf6a5ec),四仓 --all 全绿。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -35,7 +35,7 @@ feat: 迁移珊瑚橙主题体系并新增认证基础组件(ADR-005)
|
||||
|
||||
## 凭证防泄漏检查(ADR-021)
|
||||
|
||||
两层检查共用同一规则表,单一来源为各仓入库的 `scripts/check-secrets.sh`(纯 shell,零外部依赖;三仓副本内容同构,调整规则时三仓同步提交):
|
||||
两层检查共用同一规则表,单一来源为各仓入库的 `scripts/check-secrets.sh`(纯 shell,零外部依赖;**四仓**副本内容同构,调整规则时四仓同步提交——`patbond-ai` 是唯一持有 AI 服务商凭证的仓,尤其不可漏):
|
||||
|
||||
1. **本地 pre-commit(推荐,每人每仓启用一次)**:
|
||||
|
||||
@@ -45,9 +45,9 @@ feat: 迁移珊瑚橙主题体系并新增认证基础组件(ADR-005)
|
||||
```
|
||||
|
||||
之后每次 `git commit` 自动扫描暂存区内容与文件名。注意 `core.hooksPath` 会整体接管 hooks 目录(当前三仓无其他自定义 hook)。`git commit --no-verify` 可绕过,但仅限确认误报时使用——CI 兜底仍会拦。
|
||||
2. **CI 兜底(强制)**:三仓 `ci.yml` 在 checkout 后的首个 step 运行同一脚本的 `--all` 模式,对全部已跟踪文件扫描(本次 push 变更文件的超集),命中即红,禁止合入。
|
||||
2. **CI 兜底(强制)**:四仓 `ci.yml` 在 checkout 后的首个 step 运行同一脚本的 `--all` 模式,对全部已跟踪文件扫描(本次 push 变更文件的超集),命中即红,禁止合入。
|
||||
|
||||
规则覆盖(细节以脚本内规则表为准,不在文档重复维护,避免两处漂移):云厂商 AccessKey 形态(AWS/腾讯云/阿里云前缀)、**AI 服务商密钥形态(Anthropic `sk-ant-`、OpenAI `sk-` / `sk-proj-` / `sk-svcacct-`)**、MinIO 默认凭证、独立成行的私钥 PEM 头、access/secret key **与 api key/secret/token、auth token** 与 JWT/签名密钥的实值赋值、配置类文件中非 `${}` 注入形态的数据库口令、`.env`/credentials/密钥导出 CSV 文件本体误入版本库。允许清单:`${}` 注入形态、占位值(changeme、your-xxx、`<占位>`、`xxx` 等)与明显示例值——配置真实值仍只允许存在于被 gitignore 的文件中,占位只进 `.sample`。
|
||||
规则覆盖(细节以脚本内规则表为准,不在文档重复维护,避免两处漂移):云厂商 AccessKey 形态(AWS/腾讯云/阿里云前缀)、**AI 服务商密钥形态(Anthropic `sk-ant-`、OpenAI `sk-` / `sk-proj-` / `sk-svcacct-`)**、MinIO 默认凭证、独立成行的私钥 PEM 头、access/secret key **与 api key/secret/token、auth token** 与 JWT/签名密钥的实值赋值、配置类文件中非 `${}` 注入形态的数据库口令、`.env`/credentials/密钥导出 CSV 文件本体误入版本库。允许清单:`${}` 注入形态、占位值(changeme、your-xxx、`<占位>`、`xxx` 等)与明显示例值,以及**依赖注入的传参引用**(如 `api_key=settings.api_key`——行里没有任何秘密,而做 DI 的代码到处都是这种写法)。后者收得很紧:值必须是未加引号的小写起首标识符、只允许恰好一个点且随即行尾,故三段式 JWT 与 `sk-ant-…` 这类前缀仍会被拦。配置真实值仍只允许存在于被 gitignore 的文件中,占位只进 `.sample`。
|
||||
|
||||
手动全量自查:`sh scripts/check-secrets.sh --all`(在仓库根目录执行)。
|
||||
|
||||
|
||||
@@ -2,7 +2,8 @@
|
||||
# check-secrets.sh —— 凭证防泄漏检查(ADR-021,规范见 patbond-doc docs/development/git-workflow.md)
|
||||
#
|
||||
# 规则单一来源:本地 pre-commit 与 CI 兜底跑的是同一个脚本、同一张规则表。
|
||||
# 三仓(patbond-api / patbond-flutter / patbond-doc)各存一份同构副本,改规则时三仓同步。
|
||||
# 四仓(patbond-api / patbond-flutter / patbond-doc / patbond-ai)各存一份同构副本,
|
||||
# 改规则时四仓同步提交。patbond-ai 是唯一持有 AI 服务商凭证的仓,尤其不可漏。
|
||||
#
|
||||
# 用法:
|
||||
# sh scripts/check-secrets.sh --staged # pre-commit:扫暂存区内容(经 scripts/hooks/pre-commit 调用)
|
||||
@@ -19,6 +20,15 @@ mode="${1:---all}"
|
||||
# 放行它使「讲解规则的文档」不会被规则自身拦下(如 iteration-4/08 §漏检清单)。
|
||||
ALLOW='\$\{[^}]*\}|\{\{[^}]*\}\}|changeme|change[-_]me|your[-_][a-zA-Z0-9_-]+|<[a-zA-Z0-9 ,_.-]+>|placeholder|example|sample|dummy|fake|redacted|\*\*\*|x{3,}'
|
||||
|
||||
# 依赖注入的传参不是字面量密钥:`api_key=settings.api_key`、
|
||||
# `.param("secretKey", props.getSecretKey())` 这类行里没有任何秘密,而做 DI 的
|
||||
# 代码到处都是这种写法——不放行就得在每个调用点绕过一次。
|
||||
# 收得很紧,避免把真凭证放进来:
|
||||
# * 值必须是**未加引号**的小写起首标识符引用(真密钥要么带引号,要么不是标识符形状)
|
||||
# * 只允许**恰好一个点**且随即行尾(JWT 是 a.b.c 三段,仍会被拦)
|
||||
# * 连字符不在字符集内(sk-ant-… 这类前缀仍会被拦)
|
||||
ALLOW_REFERENCE='[:=][[:space:]]*[a-z][a-zA-Z0-9_]*\.[a-z][a-zA-Z0-9_]*[[:space:]]*[,);]?$'
|
||||
|
||||
# 内容扫描跳过:本脚本与 hook 自身(含规则文本,非凭证)
|
||||
SKIP_PATHS='(^|/)scripts/(check-secrets\.sh|hooks/pre-commit)$'
|
||||
|
||||
@@ -89,6 +99,7 @@ for f in $files; do
|
||||
ci=""
|
||||
[ "$flag" = "i" ] && ci="-i"
|
||||
grep -InE $ci -e "$pat" "$tmp" 2>/dev/null | grep -viE "$ALLOW" |
|
||||
grep -vE "$ALLOW_REFERENCE" |
|
||||
sed "s|^|[$id] $f:|" >>"$viol" || true
|
||||
done
|
||||
done
|
||||
@@ -100,7 +111,7 @@ if [ -s "$viol" ]; then
|
||||
处置:
|
||||
1. 若是真实凭证:先去云控制台轮换/禁用该密钥,再从提交中移除;
|
||||
2. 若是误报:改用 ${} 注入或占位值(changeme / your-xxx / <占位>),
|
||||
或与团队确认后调整三仓同构的 scripts/check-secrets.sh 规则表。
|
||||
或与团队确认后调整四仓同构的 scripts/check-secrets.sh 规则表。
|
||||
敏感信息只允许存在于被 gitignore 的文件或 .sample 占位中(git-workflow.md)。
|
||||
MSG
|
||||
exit 1
|
||||
|
||||
Reference in New Issue
Block a user