Skip to content

ci: pin all GitHub Actions to commit SHAs so a moved tag cannot inject code into workflows that hold secrets or write permissions - #1572

Merged
vbreuss merged 1 commit into
mainfrom
ci/pin-github-actions
Nov 5, 2026
Merged

vbreuss merged 1 commit into
mainfrom
ci/pin-github-actions

Conversation

@vbreuss

@vbreuss vbreuss commented Nov 5, 2026

Copy link
Copy Markdown
Member

Renovate's pinGitHubActionDigests preset keeps the pins up to date.

…t code into workflows that hold secrets or write permissions

Renovate's pinGitHubActionDigests preset keeps the pins up to date.
@vbreuss vbreuss self-assigned this Nov 5, 2026
Copilot AI balanced review requested due to automatic review settings November 5, 2026 15:43
@vbreuss
vbreuss enabled auto-merge (squash) November 5, 2026 15:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

There are no active review issues for the GitHub Actions pinning and Renovate digest-maintenance changes.

No open issues 🎉

What changed in this PR

This PR hardens System.IO.Abstractions workflows against moved action tags by pinning GitHub Actions to commit SHAs, reducing risk in jobs with secrets or write permissions.

Changes:

  • Replace tag-based action references with full commit SHAs in CI, build, and release workflows.
  • Retain version comments alongside pins for readability.
  • Enable Renovate’s helpers:pinGitHubActionDigests preset to maintain pins automatically.
File Description
renovate.json5 Enable automated GitHub Actions digest pinning.
.github/​workflows/​ci.yml Pin actions used for testing, analysis, and result publishing.
.github/​workflows/​build.yml Pin actions used for builds, package publishing, and releases.

Review effort: Balanced

@sonarqubecloud

sonarqubecloud Bot commented Nov 5, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Test Results

    40 files      40 suites   18s ⏱️
 1 177 tests  1 176 ✅     1 💤 0 ❌
14 732 runs  13 616 ✅ 1 116 💤 0 ❌

Results for commit 3cce4fb.

♻️ This comment has been updated with latest results.

@vbreuss
vbreuss merged commit f89d72b into main Nov 5, 2026
18 of 19 checks passed
@mergify

mergify Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

This pull request is merged.

@github-actions

Copy link
Copy Markdown

This is addressed in release v23.0.0.

@github-actions github-actions Bot added the state: released Issues that are released label Nov 10, 2026
This was referenced Nov 11, 2026
This was referenced Nov 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state: released Issues that are released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants