Skip to content

Security hardening for GitHub Actions - #4801

Open
madrob wants to merge 2 commits into
ml-explore:mainfrom
madrob:actions-security
Open

madrob wants to merge 2 commits into
ml-explore:mainfrom
madrob:actions-security

Conversation

@madrob

@madrob madrob commented Oct 30, 2026

Copy link
Copy Markdown
Contributor
  • Replace direct input expansion with env usage to prevent template injection.
  • Add lock file to pin actions and ensure local actions are always read from same repository

@madrob
madrob requested a review from zcbenz October 30, 2026 13:39

@zcbenz zcbenz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding the lock file.

  • Replace direct input expansion with env usage to prevent template injection.

Can you elaborate on this? The inputs are either hard-coded in files, or have to be manually entered by maintainers when manually invoke a workflow, there is no place for injection from untrusted parties.

This branch has not been deployed

No deployments
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