Skip to content

security: pin actions/attest-build-provenance@v1 to commit SHA#3

Merged
stackedsax merged 1 commit into
mainfrom
security/pin-action-shas
Jun 17, 2026
Merged

security: pin actions/attest-build-provenance@v1 to commit SHA#3
stackedsax merged 1 commit into
mainfrom
security/pin-action-shas

Conversation

@pavlovic-ivan

@pavlovic-ivan pavlovic-ivan commented May 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Pins the last remaining unpinned uses: reference in this repo — actions/attest-build-provenance@v1 in .github/workflows/docker-publish.yml — to a 40-character commit SHA.

Context

This PR was originally a broader pin sweep against the v4-era actions in this repo. Since then, #5 landed and brought every other action to a SHA-pinned, node24-compatible version. The only stragglers left unpinned were actions/attest-build-provenance@v1 at line 63 of docker-publish.yml.

The branch has been reset to the current main so the PR now contains only the meaningful 1-line delta.

Diff

-        uses: actions/attest-build-provenance@v1
+        uses: actions/attest-build-provenance@ef244123eb79f2f7a7e75d99086184180e6d0018 # v1.4.4

Notes

  • The pinned SHA ef244123… is what v1 currently resolves to (tagged v1.4.4).
  • actions/attest-build-provenance v4.1.0 is the latest major (composite action — no Node-runtime concern). A separate PR could bump to v4 if desired; out of scope here to keep this change minimal.
  • No functional change expected — same code, just pinned.

@netlify

netlify Bot commented May 27, 2026

Copy link
Copy Markdown

Deploy Preview for mlops-studio ready!

Name Link
🔨 Latest commit fd919c0
🔍 Latest deploy log https://app.netlify.com/projects/mlops-studio/deploys/6a32eb6bc1c3240008c0e7f7
😎 Deploy Preview https://deploy-preview-3--mlops-studio.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@pavlovic-ivan
pavlovic-ivan marked this pull request as draft May 27, 2026 12:36
@pavlovic-ivan
pavlovic-ivan marked this pull request as ready for review May 27, 2026 13:54
@pavlovic-ivan
pavlovic-ivan requested a review from stackedsax May 27, 2026 14:37
stackedsax
stackedsax previously approved these changes Jun 17, 2026

@stackedsax stackedsax 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.

👍 lgtm

@pavlovic-ivan
pavlovic-ivan force-pushed the security/pin-action-shas branch from 97dcec9 to 2e71bd6 Compare June 17, 2026 18:01
@pavlovic-ivan pavlovic-ivan changed the title security: pin GitHub Actions references to commit SHAs security: pin actions/attest-build-provenance@v1 to commit SHA Jun 17, 2026
Last remaining unpinned uses: ref after #5 brought everything else
to SHA-pinned node24 versions. Pinned at v1.4.4 (the SHA the v1
floating tag currently resolves to); a separate PR can bump to v4
if desired (composite, no node-runtime concern).
@pavlovic-ivan
pavlovic-ivan force-pushed the security/pin-action-shas branch from 2e71bd6 to fd919c0 Compare June 17, 2026 18:46
@pavlovic-ivan
pavlovic-ivan requested a review from stackedsax June 17, 2026 18:50

@stackedsax stackedsax 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.

I spoke too soon earlier. 👍

@stackedsax
stackedsax merged commit df2ee3b into main Jun 17, 2026
12 checks passed
@stackedsax
stackedsax deleted the security/pin-action-shas branch June 17, 2026 22:40
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