-
Notifications
You must be signed in to change notification settings - Fork 12
chore(deps): bump docker/setup-buildx-action from 3 to 4 #120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -55,7 +55,7 @@ jobs: | |
| uses: docker/setup-qemu-action@v3 | ||
|
|
||
| - name: Set up Docker Buildx | ||
| uses: docker/setup-buildx-action@v3 | ||
| uses: docker/setup-buildx-action@v4 | ||
|
|
||
| - name: Run Buildx | ||
| run: | | ||
|
|
@@ -95,7 +95,7 @@ jobs: | |
| uses: docker/setup-qemu-action@v3 | ||
|
|
||
| - name: Set up Docker Buildx | ||
| uses: docker/setup-buildx-action@v3 | ||
| uses: docker/setup-buildx-action@v4 | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Semgrep identified an issue in your code:
More details about this
A plausible attack looks like this:
To resolve this comment: ✨ Commit fix suggestion
💬 Ignore this findingReply with Semgrep commands to ignore this finding.
Alternatively, triage in Semgrep AppSec Platform to ignore the finding created by github-actions-mutable-action-tag. 🛟 Help? Slack #semgrep-help or go/semgrep-help. Resolution Options:
You can view more details about this finding in the Semgrep AppSec Platform. |
||
|
|
||
| - name: Run Buildx | ||
| run: | | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -69,7 +69,7 @@ jobs: | |||||
| uses: docker/setup-qemu-action@v3 | ||||||
|
|
||||||
| - name: Set up Docker Buildx | ||||||
| uses: docker/setup-buildx-action@v3 | ||||||
| uses: docker/setup-buildx-action@v4 | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Semgrep identified an issue in your code:
More details about this
A plausible attack looks like this:
Because the reference is To resolve this comment: ✨ Commit fix suggestion
Suggested change
View step-by-step instructions
💬 Ignore this findingReply with Semgrep commands to ignore this finding.
Alternatively, triage in Semgrep AppSec Platform to ignore the finding created by github-actions-mutable-action-tag. 🛟 Help? Slack #semgrep-help or go/semgrep-help. Resolution Options:
You can view more details about this finding in the Semgrep AppSec Platform. |
||||||
|
|
||||||
| - name: Run Buildx | ||||||
| run: | | ||||||
|
|
@@ -110,7 +110,7 @@ jobs: | |||||
| uses: docker/setup-qemu-action@v3 | ||||||
|
|
||||||
| - name: Set up Docker Buildx | ||||||
| uses: docker/setup-buildx-action@v3 | ||||||
| uses: docker/setup-buildx-action@v4 | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Semgrep identified an issue in your code:
More details about this
A plausible attack looks like this:
Because the reference is To resolve this comment: ✨ Commit fix suggestion
Suggested change
View step-by-step instructions
💬 Ignore this findingReply with Semgrep commands to ignore this finding.
Alternatively, triage in Semgrep AppSec Platform to ignore the finding created by github-actions-mutable-action-tag. 🛟 Help? Slack #semgrep-help or go/semgrep-help. Resolution Options:
You can view more details about this finding in the Semgrep AppSec Platform. |
||||||
|
|
||||||
| - name: Run Buildx | ||||||
| run: | | ||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Semgrep identified an issue in your code:
docker/setup-buildx-actionis referenced by the mutablev4tag, so future workflow runs may execute different code if that tag is repointed. A compromised action release could tamper with the Buildx setup and the later Docker build in this job.More details about this
docker/setup-buildx-action@v4is pulled by the mutablev4tag, so this workflow will run whatever commit the action owner later pointsv4to. In this job, that action runs right beforedocker buildx create --name builder --bootstrap --useanddocker buildx build, so a repointed tag could change how the builder is installed or execute attacker-controlled code on the GitHub runner.A plausible attack looks like this:
docker/setup-buildx-actionrepository or a maintainer account and moves thev4tag to a malicious commit.container_tests_glibcjob starts and executesuses: docker/setup-buildx-action@v4, fetching that new commit automatically.docker buildx buildcommand, for example by altering the builder used forlinux/${{ matrix.linux_arch }}images or by exfiltrating repository data before theupload-artifactstep publishes the build output.Because the reference is not a full 40-character commit SHA, the code that runs here can change without any PR in this repository.
To resolve this comment:
✨ Commit fix suggestion
usesline fordocker/setup-buildx-action.uses: docker/setup-buildx-action@<full-40-char-sha> # v4.v4release you intend to keep, not a branch name or short hash. Pinning to a commit prevents the action owner from moving the reference to different code later.💬 Ignore this finding
Reply with Semgrep commands to ignore this finding.
/fp <comment>for false positive/ar <comment>for acceptable risk/other <comment>for all other reasonsAlternatively, triage in Semgrep AppSec Platform to ignore the finding created by github-actions-mutable-action-tag.
🛟 Help? Slack #semgrep-help or go/semgrep-help.
Resolution Options:
/fp $reason(if security gap doesn’t exist)/ar $reason(if gap is valid but intentional; add mitigations/monitoring)/other $reason(e.g., test-only)You can view more details about this finding in the Semgrep AppSec Platform.