Repository navigation
fix: refresh transitive undici dependency - #338
Merged
Merged
Conversation
codeforester
force-pushed
the
security/334-20261005-refresh-undici
branch
from
October 5, 2026 17:33
558249d to
dd93e85
Compare
codeforester
changed the base branch from
ci/327-20261005-add-workspace-update-scenarios
to
main
October 5, 2026 17:33
codeforester
commented
Oct 6, 2026
codeforester
left a comment
Collaborator
Author
There was a problem hiding this comment.
Reviewed head dd93e85 against #334. No findings. This is a lockfile-only bump of the transitive dev dependency undici 7.29.0 → 7.29.1 (via jsdom), with integrity updated, which clears GHSA-3wwx-pv8p-q78v without suppressing the audit. CI is green (7/7). Ready to merge, and merge it first: #335, #336 and #337 all fail validate/validate-ubuntu on exactly this advisory, so they'll go green once this lands and they're refreshed.
This was referenced Oct 6, 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.
Summary
Refresh the locked transitive
undicidependency pulled byjsdomfrom 7.29.0 to patched 7.29.1. This restores the required demo-console audit gate without suppressing advisories or changing production dependencies.Issue
Fixes #334
Validation
npm ci --ignore-scripts(139 packages; 0 vulnerabilities)npm ls undici --allshowsjsdom@28.1.0 -> undici@7.29.1npm run validate(6 unit tests passed)npm run buildpassednpm run auditpassed with 0 vulnerabilitiesgit diff --checkNotes
This is the first, independent train item because the existing required Ubuntu audit gate was failing on the unpatched transitive dependency. The dependency is development-only; this change does not claim a demonstrated shipped-browser exploit. The full repository validator was also attempted locally; its first run used the machine Node 14 toolchain and its Node 22 rerun was blocked by the host Gradle native-platform library, while the affected frontend gates passed under the CI-matching Node toolchain.
Current audit follow-up
The current npm advisory database also flags transitive
source-map-js@1.2.1(GHSA-68fv-2mgg-jv7q). The same lockfile change adds a narrow npm override to^1.2.2;npm ci,npm run validate,npm run build, andnpm run auditpass with zero vulnerabilities.