Skip to content

ci: pin third-party actions (github-push-action, free-disk-space) to full commit SHAs - #9575

Merged
okatu-loli merged 1 commit into
AlistGo:mainfrom
kobihikri:harden/pin-thirdparty-actions
Aug 5, 2026
Merged

ci: pin third-party actions (github-push-action, free-disk-space) to full commit SHAs#9575
okatu-loli merged 1 commit into
AlistGo:mainfrom
kobihikri:harden/pin-thirdparty-actions

Conversation

@kobihikri

Copy link
Copy Markdown
Contributor

What

Pin the third-party actions referenced by @master/@main to their current commit SHA (tag kept in a comment):

Action Where
ad-m/github-push-action@master auto_lang.yml, beta_release.yml, release.yml
jlumbroso/free-disk-space@main release.yml

Why

@master/@main are moving refs. In release.yml, github-push-action is passed
secrets.MY_TOKEN (a write-scoped token). If that action's master were moved (compromise or an
accidental change), the new code would run with the token — the class of issue behind the 2025
tj-actions/changed-files incident. Pinning to a SHA removes that.

Scope / safety

Behaviour unchanged — each SHA is the current tip of that action's default branch. Only third-party
actions touched. Signed-off. Per GitHub's pin-to-SHA guidance.

AI-assisted; I verified the workflows, the token exposure, and the pinned SHAs myself.

Pin the third-party actions referenced by mutable @master/@main tags to their
current commit SHA (tag kept in a trailing comment):
- ad-m/github-push-action (auto_lang, beta_release, release) — passed secrets.MY_TOKEN
- jlumbroso/free-disk-space (release)

release.yml hands github-push-action a write-scoped token, so a moved tag there
would run unreviewed code with it. Behaviour unchanged; per GitHub's guidance to
pin actions to a full-length commit SHA.

Signed-off-by: Kobi Hikri <kobi.hikri@gmail.com>
@okatu-loli
okatu-loli merged commit 8d77000 into AlistGo:main Aug 5, 2026
12 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.

3 participants