Skip to content

agents: add GitHub Actions workflow security review rules - #4359

Merged
rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:audit_gha_injection_vulnerabilities
Oct 26, 2026
Merged

rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:audit_gha_injection_vulnerabilities

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

GitHub Actions workflows and scripts can introduce command, flag, and
environment injection vulnerabilities when handling untrusted inputs
from pull requests, comments, and git refs. Without explicit review
criteria, code review audits may overlook these attack vectors.

Add security review rules and audit checks covering expression
interpolation in run: and script: blocks, untrusted git ref names,
CLI option-flag injection, pre-mutation actor authorization,
single-line $GITHUB_OUTPUT and $GITHUB_ENV writes, and untrusted
stdout suppression.

GitHub Actions workflows and scripts can introduce command, flag, and
environment injection vulnerabilities when handling untrusted inputs
from pull requests, comments, and git refs. Without explicit review
criteria, code review audits may overlook these attack vectors.

Add security review rules and audit checks covering expression
interpolation in `run:` and `script:` blocks, untrusted git ref names,
CLI option-flag injection, pre-mutation actor authorization,
single-line `$GITHUB_OUTPUT` and `$GITHUB_ENV` writes, and untrusted
stdout suppression.
@rickeylev

Copy link
Copy Markdown
Collaborator Author

AI Disclosure

  • Oversight: semi-autonomous — This PR was authored by an AI coding assistant under human direction and review.
  • Comment Handling: Review comments on this PR are monitored and reviewed by the human author, who may use an AI assistant to draft responses or fixes before approving and posting them.

This is an automatically generated agent comment.

@rickeylev
rickeylev merged commit 2984905 into bazel-contrib:main Oct 26, 2026
13 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.

1 participant