Bump actions/upload-artifact from 4 to 7#453
Closed
dependabot[bot] wants to merge 1 commit intomainfrom
Closed
Conversation
dependabot
bot
added
Changed
🤖 Cursor Dependency AnalysisSupply-Chain Malware ReviewScanning the repo for Verdict: benign Why this is acceptable
Actionable caveat (ops, not malware): v6+ needs a runner ≥ 2.327.1 if you use self-hosted runners; GitHub-hosted runners are fine. No extra steps unless you use Scanner vs. interpretation: The tool flagged many “codepoint decoder” lines; those are consistent with benign bundled JS and disagree with a malware reading unless paired with IOC/confusable findings—which are absent here. Compatibility AnalysisSearching the repo for 1) Where
|
| File | Role |
|---|---|
.github/workflows/deploy-review.yml |
Uploads the build/ directory after npm run build (name: build, path: build). |
.github/workflows/dependabot-cursor-review.yml |
Uploads malware-scan outputs (malware_scan_report.json, malware_scan_summary.md, upstream_changed_files.txt) with if-no-files-found: warn. |
No app code or package manifests reference this action.
2) Usage vs. changed APIs (v4 → v7)
From .upstream-dependency/action.yml and the release notes:
archive(v7) — Optional; default istrue(zipped archive). Your workflows do not setarchive: false, so you avoid the new single-file / ignore-namebehavior.- Inputs you use —
name,path, andif-no-files-foundare still defined; defaults and semantics for normal directory and multi-file uploads are unchanged for the defaultarchive: truepath. - Runtime — The action runs with
runs.using: node24(v6+). That affects the runner, not yournode:20-alpinejob container indeploy-review(the action process is separate).
deploy-review.yml already pairs actions/download-artifact@v7 with the same artifact name, so major-version alignment is consistent.
3) Risks / unknowns
- Self-hosted runners — v6+ requires runner ≥ 2.327.1 for Node 24–based actions. GitHub-hosted
ubuntu-latestis fine; old self-hosted runners could fail until upgraded. - GitHub Enterprise Server — If you ever run these workflows on GHES, confirm your instance supports the action/runtime versions you pin (release notes call this out for older stacks).
- Internal ESM change (v7) — Implementation detail of the action; no change required in your YAML unless you relied on undocumented behavior (you do not).
4) Recommendation
Merge for typical GitHub-hosted usage.
Use merge-with-caveats only if you rely on self-hosted runners: verify they are at least 2.327.1 before merging, or hold until those runners are updated.
Hold is not warranted for hosted GitHub Actions based on current usage and inputs.
Malware Scan Summary
- Status: warn
- Warn only mode:
true - Changed upstream files scanned:
35 - Resolution strategy:
tag_range - Changed node/vendor paths:
0 - Changed lockfiles:
1 - Resolved upstream range:
ea165f8d65b6e75b540449e92b4886f43607fa02..bbbca2ddaa5d8feaa63e36b76fdaad77386f024f - Resolved refs: from=
ea165f8d65b6e75b540449e92b4886f43607fa02to=bbbca2ddaa5d8feaa63e36b76fdaad77386f024f - Unicode findings (post-allowlist):
0 - Confusable findings (post-allowlist):
0 - IOC findings (post-allowlist):
0 - Heuristic findings (post-allowlist):
530
Top findings
dist/upload/index.js:933codepoint_decoder ::: str.charCodeAt(0);dist/upload/index.js:3414codepoint_decoder ::decTable[encTable[i].charCodeAt(0)] = i;dist/upload/index.js:3416codepoint_decoder ::decTable["-".charCodeAt(0)] = encTable.indexOf("+");dist/upload/index.js:3417codepoint_decoder ::decTable["_".charCodeAt(0)] = encTable.indexOf("/");dist/upload/index.js:3444codepoint_decoder ::b = decTable[base64Str.charCodeAt(i)];dist/upload/index.js:19575codepoint_decoder ::for(var i = 0, L = bstr.length; i < L;) C = (C>>>8) ^ T0[(C^bstr.charCodeAt(i++))&0xFF];dist/upload/index.js:19597codepoint_decoder ::c = str.charCodeAt(i++);dist/upload/index.js:19604codepoint_decoder ::c = (c&1023)+64; d = str.charCodeAt(i++)&1023;dist/upload/index.js:20100codepoint_decoder ::hash = ((hash << 5) - hash) + namespace.charCodeAt(i);dist/upload/index.js:25872codepoint_decoder ::var c = r.charCodeAt(r.length - 1);dist/upload/index.js:36617codepoint_decoder ::: str.charCodeAt(0);dist/upload/index.js:39331codepoint_decoder ::var c = r.charCodeAt(r.length - 1);dist/upload/index.js:39990codepoint_decoder ::const ZERO_OFFSET = '0'.charCodeAt(0)dist/upload/index.js:40756codepoint_decoder ::result += String.fromCodePoint(this.codePoint)dist/upload/index.js:40816codepoint_decoder ::result += String.fromCodePoint(this.codePoint)dist/upload/index.js:44693codepoint_decoder ::const code = this.code = key.charCodeAt(index)dist/upload/index.js:44717codepoint_decoder ::const code = key.charCodeAt(index)dist/upload/index.js:45406codepoint_decoder ::if (!isTokenCharCode(characters.charCodeAt(i))) {dist/upload/index.js:51396codepoint_decoder ::for (let i = 'A'.charCodeAt(0); i <= 'Z'.charCodeAt(0); i++) {dist/upload/index.js:54487codepoint_decoder ::const charCode = attributeValue.charCodeAt(0)
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 6 to 7. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v6...v7) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
bot
changed the title
dependabot
bot
force-pushed
the
dependabot/github_actions/actions/upload-artifact-7
branch
from
6841b07 to
8c4eecc
Compare
Contributor
|
@dependabot rebase |
Contributor
Author
|
Looks like actions/upload-artifact is up-to-date now, so this is no longer needed. |
dependabot
bot
deleted the
dependabot/github_actions/actions/upload-artifact-7
branch
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
1 participant
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.
Bumps actions/upload-artifact from 4 to 7.
Release notes
Sourced from actions/upload-artifact's releases.
... (truncated)
Commits
bbbca2dSupport direct file uploads (#764)589182cUpgrade the module to ESM and bump dependencies (#762)47309c9Merge pull request #754 from actions/Link-/add-proxy-integration-tests02a8460Add proxy integration testb7c566aMerge pull request #745 from actions/upload-artifact-v6-releasee516bc8docs: correct description of Node.js 24 support in READMEddc45eddocs: update README to correct action name for Node.js 24 support615b319chore: release v6.0.0 for Node.js 24 support017748bMerge pull request #744 from actions/fix-storage-blob38d4c79chore: rebuild distNote
Low Risk
Low risk: this only updates the
actions/upload-artifactversion used in GitHub Actions workflows, with no application code changes; the main risk is workflow/runtime incompatibility if the new action behavior or Node runtime requirements differ.Overview
Updates GitHub Actions workflows to use
actions/upload-artifact@v7instead of older major versions.This affects artifact uploads in the Dependabot Cursor malware-scan workflow and the PR deploy/review build workflow, keeping artifact publishing aligned with the latest
upload-artifactaction.Written by Cursor Bugbot for commit 8c4eecc. This will update automatically on new commits. Configure here.