Skip to content

fix: refresh transitive undici dependency - #338

Merged
codeforester merged 2 commits into
mainfrom
security/334-20261005-refresh-undici
Oct 6, 2026
Merged

codeforester merged 2 commits into
mainfrom
security/334-20261005-refresh-undici

Conversation

@codeforester

@codeforester codeforester commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Refresh the locked transitive undici dependency pulled by jsdom from 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

  • Fresh Node 22.22.0/npm 10.9.4 npm ci --ignore-scripts (139 packages; 0 vulnerabilities)
  • npm ls undici --all shows jsdom@28.1.0 -> undici@7.29.1
  • npm run validate (6 unit tests passed)
  • npm run build passed
  • npm run audit passed with 0 vulnerabilities
  • Registry tarball SHA-512 matches the committed lockfile integrity
  • git diff --check

Notes

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, and npm run audit pass with zero vulnerabilities.

@codeforester
codeforester force-pushed the security/334-20261005-refresh-undici branch from 558249d to dd93e85 Compare October 5, 2026 17:33
@codeforester
codeforester changed the base branch from ci/327-20261005-add-workspace-update-scenarios to main October 5, 2026 17:33

@codeforester codeforester left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@codeforester
codeforester merged commit fb662f0 into main Oct 6, 2026
8 checks passed
@codeforester
codeforester deleted the security/334-20261005-refresh-undici branch October 6, 2026 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refresh the locked Undici development dependency to resolve current advisories

1 participant