Skip to content

security(deps-dev): override undici to 7.29.0 and bump brace-expansion - #248

Merged
kellenmurphy merged 1 commit into
mainfrom
security/undici-brace-expansion
Aug 3, 2026
Merged

security(deps-dev): override undici to 7.29.0 and bump brace-expansion#248
kellenmurphy merged 1 commit into
mainfrom
security/undici-brace-expansion

Conversation

@kellenmurphy

Copy link
Copy Markdown
Owner

Why

The Dependency Audit job on main is failing on six advisories, none of which Dependabot can fix on its own:

What

  • Add an npm override scoped under miniflare forcing undici: ^7.29.0 (the first patched version). Scoping matters: jsdom resolves undici 8.9.0, a different major, and a global override would downgrade it.
  • npm update brace-expansion → 5.0.9.

The override should be dropped once wrangler ships a miniflare requiring undici >= 7.29.0.

Verification

  • npm audit: 0 vulnerabilities
  • npx tsc --noEmit, npx eslint . --quiet: clean
  • npm run coverage: 100% statements/branches/functions/lines

Resolves five open Dependabot alerts on undici (GHSA-4cwx-7wf7-3272,
GHSA-8xcm-r25x-g524, GHSA-m8rv-5g2x-5cg5, GHSA-jr45-8vmc-qm54,
GHSA-v3r7-h72x-cjcm) and GHSA-rgw5-rvv9-x895 on brace-expansion, which
were failing the Dependency Audit job on main.

miniflare pins undici to exactly 7.28.0 (including its latest release),
so neither npm audit fix nor Dependabot can bump it transitively; the
override is scoped under miniflare so jsdom's undici 8.x is unaffected.
Drop the override once wrangler ships a miniflare that requires
undici >= 7.29.0.
@kellenmurphy
kellenmurphy enabled auto-merge August 3, 2026 22:29
@kellenmurphy
kellenmurphy merged commit 26e9fc3 into main Aug 3, 2026
9 checks passed
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

1 participant