Skip to content

FIX: bump transitive ws to 8.21.0 to address GHSA-58qx-3vcg-4xpx#1873

Merged
romanlutz merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz/dependabot-fix-ws
Jun 1, 2026
Merged

FIX: bump transitive ws to 8.21.0 to address GHSA-58qx-3vcg-4xpx#1873
romanlutz merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz/dependabot-fix-ws

Conversation

@romanlutz
Copy link
Copy Markdown
Contributor

The frontend lockfile pulled in ws@8.19.0 transitively via jest-environment-jsdom -> jsdom@20.0.3, which is vulnerable to uninitialized memory disclosure in WebSocket.close() when a TypedArray is passed as the reason argument (GHSA-58qx-3vcg-4xpx / CVE-2026-45736, medium severity, fixed in 8.20.1).

GitHub auto-dismissed alert #170 because the dependency is transitive and dev-only, but the vulnerable version was still resolved in package-lock.json. This PR adds ws to the existing overrides block in frontend/package.json and pins it to 8.21.0 (the latest 8.x, satisfying jsdom's ws@^8.11.0 range).

Verification

  • npm install resolves ws@8.21.0 overridden (confirmed via npm ls ws).
  • npm run type-check passes.
  • npm run lint passes.
  • Lockfile is consistent with package.json (no drift after npm install).
  • Latest main merged in cleanly with no conflicts.

romanlutz and others added 2 commits May 30, 2026 06:38
The frontend lockfile pulled in ws@8.19.0 transitively via
jest-environment-jsdom -> jsdom@20.0.3, which is vulnerable to
uninitialized memory disclosure in WebSocket.close() when a TypedArray
is passed as the reason argument (GHSA-58qx-3vcg-4xpx / CVE-2026-45736,
medium severity, fixed in 8.20.1).

GitHub auto-dismissed alert microsoft#170 because the dependency is transitive
and dev-only, but the vulnerable version was still resolved in
package-lock.json. Add ws to the existing `overrides` block and pin
it to 8.21.0 (the latest 8.x, satisfying jsdom's ws@^8.11.0 range).

Verified locally: npm install resolves ws@8.21.0 (overridden);
type-check, lint, and the full jest suite all pass.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@romanlutz romanlutz added this pull request to the merge queue Jun 1, 2026
Merged via the queue into microsoft:main with commit b7b42f0 Jun 1, 2026
48 checks passed
@romanlutz romanlutz deleted the romanlutz/dependabot-fix-ws branch June 1, 2026 19:36
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.

2 participants