Skip to content

[RUN-5043] Pin container images and GitHub Actions by digest - #65

Merged
luismalamoc merged 2 commits into
mainfrom
feature/RUN-5043-pin-digests
Oct 20, 2026
Merged

luismalamoc merged 2 commits into
mainfrom
feature/RUN-5043-pin-digests

Conversation

@luismalamoc

Copy link
Copy Markdown
Contributor

Summary

  • Pin third-party GitHub Actions uses: to a commit SHA, with the version in a comment.

Exceptions

  • Reusable workflows we own stay on @main (rundeck-plugins/.github snyk scan and rundeck/rundeck prepare-community-pr).

Test plan

  • CI on this pull request

@luismalamoc
luismalamoc requested review from a team and a balanced review from Copilot October 19, 2026 15:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Upstream tags confirm all three SHA-to-version mappings, and the changes introduce no identified functional defects.

No open issues 🎉

What changed in this PR

This PR hardens build and release automation by pinning third-party GitHub Actions to immutable commit SHAs.

Changes:

  • Pins checkout, Java setup, and artifact upload actions.
  • Adds version comments without changing action inputs.
File Description
.github/​workflows/​release.yml Pins checkout and Java setup actions.
.github/​workflows/​gradle.yml Pins checkout, Java setup, and artifact upload actions.

Review effort: Balanced

@Jesus-Osuna-M Jesus-Osuna-M left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving RUN-5043. Workflow-only change: checkout and Java setup go through the digest-pinned composite action. Snyk is red on existing dependency findings (not introduced by this diff).

@luismalamoc
luismalamoc merged commit 724692e into main Oct 20, 2026
3 of 6 checks passed
@luismalamoc
luismalamoc deleted the feature/RUN-5043-pin-digests branch October 20, 2026 14:10
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