Skip to content

ci: scan pull requests for credentials and injection with ThreatCrush - #190

Open
ralyodio wants to merge 1 commit into
temple-compute:mainfrom
ralyodio:threatcrush-scan
Open

ci: scan pull requests for credentials and injection with ThreatCrush#190
ralyodio wants to merge 1 commit into
temple-compute:mainfrom
ralyodio:threatcrush-scan

Conversation

@ralyodio

@ralyodio ralyodio commented Sep 2, 2026

Copy link
Copy Markdown

Adds one workflow. On each pull request it scans the checked-out repository for
hardcoded credentials, injection, SSRF and unsafe deserialisation.

  • .github/workflows/threatcrush-scan.yml

Findings go to the Security tab and a pull request comment. If you would rather
not grant those write scopes, say so and I will send the contents: read build:
same scan, findings in the job summary and a SARIF artifact, and the two steps
that need a write scope removed from the file rather than switched off.

Report-only. failOn is empty, so findings never fail the build. An install
or scan failure does fail the job: a scanner that reports clean when it did not
run is worse than no scanner.

Pre-existing findings. The report leads with findings in the files the pull
request changes and folds the rest of the repository behind a <details> summary,
so an existing backlog is visible without being posted at the author of an
unrelated change. Anything intentional can be excluded with a .threatcrushignore
or a // threatcrush-disable-next-line <rule-id> comment.

Scope: it scans the whole checked-out repository, not only the diff.

Supply chain. Pinned to @profullstack/threatcrush@0.11.8; the tarball is hashed and checked against
a value in the workflow before install (npm view it yourself), installed with
--ignore-scripts, actions pinned to commit SHAs, and it runs on pull_request
rather than pull_request_target.

Asked first in #189.

Disclosure: I maintain ThreatCrush;
MIT and free. Written with AI assistance. Closing this is a fine answer and I
will not send another.

Signed-off-by: Anthony Ettinger <anthony@chovy.com>
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