diff --git a/scripts/check-secrets.sh b/scripts/check-secrets.sh index 8256a57..ccfe236 100755 --- a/scripts/check-secrets.sh +++ b/scripts/check-secrets.sh @@ -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