fix #156 - bug: Unpinned tag for a non-immutable Action in workflow#219
Open
fantonangeli wants to merge 1 commit into
Conversation
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
✅ Deploy Preview for swf-editor canceled.
|
Contributor
There was a problem hiding this comment.
Pull request overview
Pins GitHub Actions used by CI workflows to immutable commit SHAs to address the CodeQL warning about using non-immutable action tags (issue #156), reducing supply-chain risk in the repository’s CI pipelines.
Changes:
- Pin
actions/checkoutto a full commit SHA in CI workflows. - Pin
github/codeql-actionsteps (init,analyze) to a full commit SHA. - Pin additional actions in the license header workflow (
actions/setup-java,actions/cache) to full commit SHAs.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/ci_codeql.yml | Pins checkout and CodeQL actions to immutable SHAs for CodeQL workflow security. |
| .github/workflows/ci_check_license_headers.yaml | Pins checkout/setup-java/cache actions to immutable SHAs for the license-header CI job. |
| .github/workflows/ci_build.yaml | Pins checkout and upload-artifact actions to immutable SHAs for the main build/test workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
fantonangeli
added
the
github_actions
ricardozanini
approved these changes
Jul 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #156
Describe the Bug
Note: I have this security alert on my fork but I cannot see it in this fork, probably due to a lack of permissions
Unpinned tag for a non-immutable Action in workflow
Severity: warning
State: fixed
Tool: CodeQL
Description
Overview
Using a tag for a 3rd party Action that is not pinned to a commit can lead to executing an untrusted Action through a supply chain attack.
Recommendation
Pinning an action to a full length commit SHA is currently the only way to use a non-immutable action as an immutable release. Pinning to a particular SHA helps mitigate the risk of a bad actor adding a
backdoor to the action's repository, as they would need to generate a SHA-1 collision for a valid Git object payload. When selecting a SHA, you should verify it is from the action's repository and not a
repository fork.