Skip to content

Commit 824c3e2

Browse files
fix(ci): move secret-scanner Cargo.toml gate from job-level if: to step-level (#36)
`secret-scanner.yml` has had 0 successful runs since creation across all estate consumers — every run completes in 0 seconds with `conclusion=failure` and zero jobs spawned (GitHub Actions startup_failure). YAML is syntactically valid; both action SHAs exist; `gh workflow view` returns the file cleanly. ## Root cause The `rust-secrets` job has a job-level `if:` clause: ```yaml rust-secrets: runs-on: ubuntu-latest if: hashFiles('**/Cargo.toml') != '' ``` GitHub Actions does not support `hashFiles()` in **job-level** `if:` conditions. The docs say `hashFiles` is "available in the runtime environment when steps are running" — i.e. step-level only. At job-eligibility time the expression evaluator rejects the workflow, no jobs are scheduled, the run is marked as a failed startup. Wrapping in ${{ }} makes no difference. ## Fix Mirrors hyperpolymath/stapeln#36. Removes the job-level `if:` line and adds a step-level guard at the top of the existing run block: ```bash if ! find . -name Cargo.toml -not -path './target/*' -print -quit | grep -q .; then echo "No Cargo.toml found — skipping Rust secrets check" exit 0 fi ``` Same semantics (skip when no `Cargo.toml`), but at a context where the expression works. After this fix, the Secret Scanner workflow actually runs trufflehog + gitleaks + (conditionally) rust-secrets as designed.
1 parent be58ce5 commit 824c3e2

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

.github/workflows/secret-scanner.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,15 @@ jobs:
3838
# Rust-specific: Check for hardcoded crypto values
3939
rust-secrets:
4040
runs-on: ubuntu-latest
41-
if: hashFiles('**/Cargo.toml') != ''
4241
steps:
4342
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v4
4443

4544
- name: Check for hardcoded secrets in Rust
4645
run: |
46+
if ! find . -name Cargo.toml -not -path './target/*' -print -quit | grep -q .; then
47+
echo 'No Cargo.toml found — skipping Rust secrets check'
48+
exit 0
49+
fi
4750
# Patterns that suggest hardcoded secrets
4851
PATTERNS=(
4952
'const.*SECRET.*=.*"'

0 commit comments

Comments
 (0)