fix(security): force undici@^6.24.0 via npm override (clears 5 advisories)#20
Merged
Conversation
…ries)
npm audit flagged undici 5.29.0 (pulled transitively by
@actions/http-client) for 5 high/moderate advisories: unbounded
decompression, request/response smuggling, WebSocket memory exhaustion,
invalid server_max_window_bits handling, and CRLF injection. Fix
requires undici >= 6.24.0.
Add an `overrides: { undici: ^6.24.0 }` to shim/package.json and
regenerate the committed node_modules (the action runs shim/index.js
directly, so the tree must ship resolved). undici resolves to 6.26.0;
the undici-5-only @fastify/busboy dep drops out. `npm audit` now clean.
Replaces the stale #10 (cut 2026-05-13, red self-test) — same fix on
current main with a fresh, minimal lock.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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
2 participants
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.
Summary
Clears the undici advisories
npm auditflags inshim/— replaces the stale #10 with the same fix on currentmain.undici5.29.0 (pulled transitively by@actions/http-client) is vulnerable to 5 advisories (4 high / 1 moderate): unbounded decompression, request/response smuggling, WebSocket permessage-deflate memory exhaustion, invalidserver_max_window_bitshandling, CRLF injection. Fix requires undici ≥ 6.24.0.What
"overrides": { "undici": "^6.24.0" }toshim/package.json.shim/node_modules(this action runsshim/index.jsdirectly peraction.yml, so the tree ships resolved). undici resolves to 6.26.0; the undici-5-only@fastify/busboytransitive dep drops out.Verification
npm audit→ 0 vulnerabilities (was 2: 1 high + 1 moderate aggregate)npm ls undici→undici@6.26.0 overriddennode -e "require('./shim/index.js')"runs to the expected RUNNER_TEMP guard)Why not #10
#10 was cut 2026-05-13, its
Self-testcheck is red, and it predates the active v1.0.5-rc work. This is the same override on current main with a fresh, minimal lock. Closing #10 in favor of this.🤖 Generated with Claude Code