Skip to content

ci: pin GitHub Actions to commit SHAs (11)#79

Open
mirkosalvato1-ctrl wants to merge 1 commit into
LC044:masterfrom
mirkosalvato1-ctrl:pinguard/pin-actions-26711
Open

ci: pin GitHub Actions to commit SHAs (11)#79
mirkosalvato1-ctrl wants to merge 1 commit into
LC044:masterfrom
mirkosalvato1-ctrl:pinguard/pin-actions-26711

Conversation

@mirkosalvato1-ctrl

Copy link
Copy Markdown

Why

Mutable Action tags (@v4, @main) can be retagged, which is a supply-chain risk.
This PR pins third-party Actions to full commit SHAs while keeping the tag in a comment.
Official actions/* tags are left unchanged (common maintainer preference).

Pins

  • softprops/action-gh-release@v2 -> 3bb12739c298
  • docker/setup-buildx-action@v3 -> 8d2750c68a42
  • docker/login-action@v3 -> c94ce9fb4685
  • docker/build-push-action@v5 -> ca052bb54ab0
  • docker/setup-buildx-action@v3 -> 8d2750c68a42
  • docker/login-action@v3 -> c94ce9fb4685
  • docker/build-push-action@v5 -> ca052bb54ab0
  • docker/setup-buildx-action@v3 -> 8d2750c68a42
  • docker/login-action@v3 -> c94ce9fb4685
  • docker/build-push-action@v5 -> ca052bb54ab0
  • pnpm/action-setup@v3 -> a3252b78c470

Reference: GitHub docs on using third-party actions securely.

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.

Pull request overview

This PR reduces GitHub Actions supply-chain risk by pinning third-party Actions previously referenced by mutable tags (e.g., @v3, @v5, @v2) to full commit SHAs, while preserving the original tag as an inline comment for readability/auditing.

Changes:

  • Pinned pnpm/action-setup to a full commit SHA in the E2E system test workflow.
  • Pinned Docker build-related Actions (setup-buildx, login, build-push) to full commit SHAs across server/frontend/AI image build workflows.
  • Pinned softprops/action-gh-release to a full commit SHA in the CLI release workflow.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/e2e-system-tests.yml Pins pnpm/action-setup to a commit SHA while retaining the v3 marker as a comment.
.github/workflows/docker-build-push-server.yml Pins Docker Buildx/Login/Build-Push Actions to commit SHAs for server image publishing.
.github/workflows/docker-build-push-frontend.yml Pins Docker Buildx/Login/Build-Push Actions to commit SHAs for frontend image publishing.
.github/workflows/docker-build-push-ai.yml Pins Docker Buildx/Login/Build-Push Actions to commit SHAs for AI image publishing (matrix builds).
.github/workflows/build-publish-cli.yml Pins softprops/action-gh-release to a commit SHA for release creation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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