Skip to content

security(ci): allowlist incident-doc paths in gitleaks + non-block dependency-review (no GHAS)#7

Merged
skhaitan merged 1 commit into
mainfrom
security/fix-ci-gates-option-b
May 14, 2026
Merged

security(ci): allowlist incident-doc paths in gitleaks + non-block dependency-review (no GHAS)#7
skhaitan merged 1 commit into
mainfrom
security/fix-ci-gates-option-b

Conversation

@skhaitan
Copy link
Copy Markdown
Collaborator

Summary

Option-B fix for the two CI gates that block every PR after the 2026-05-09 Shai-Hulud incident docs landed:

  1. gitleaks (secret-scan.yml).gitleaks.toml allowlist did not cover the repo-root SECURITY-INCIDENT-*.md / SECURITY-AUDIT-*.md / SECURITY-EXCEPTIONS*.md files, so the custom bb-malware-blockchain-loader-marker rule fired on its own documentation. Adds those paths to the existing [allowlist].paths array.
  2. dependency-review.ymlactions/dependency-review-action errors with "Dependency review is not supported on this repository. Please ensure that Dependency graph is enabled along with GitHub Advanced Security" because GHAS is not currently enabled on private repos and is not being purchased right now. Adds continue-on-error: true at job level so the gate is non-blocking until GHAS is enabled.

Both gates remain installed and visible in PR CI runs; they just stop self-blocking every PR.

Test plan

  • After merge, open a no-op PR and confirm Gitleaks + Dependency Review checks are green (or "passed with errors" for dep-review)
  • Confirm the bb-malware-blockchain-loader-marker rule still fires on a fake commit that adds an IoC string to a non-allowlisted runtime file

🤖 Generated with Claude Code

@skhaitan skhaitan force-pushed the security/fix-ci-gates-option-b branch 4 times, most recently from eadaf68 to ead9b51 Compare May 13, 2026 23:20
@skhaitan skhaitan force-pushed the security/fix-ci-gates-option-b branch from ead9b51 to 9ef74ae Compare May 13, 2026 23:23
@skhaitan skhaitan requested a review from nitinwepro May 14, 2026 08:52
@skhaitan skhaitan merged commit b7208e2 into main May 14, 2026
2 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.

2 participants