Skip to content

Default mdm-bootstrap Node.js to patched 22.23.4 - #1274

Open
greynewell wants to merge 1 commit into
databricks:mainfrom
greynewell:security/brotli-mdm-node-default
Open

greynewell wants to merge 1 commit into
databricks:mainfrom
greynewell:security/brotli-mdm-node-default

Conversation

@greynewell

Copy link
Copy Markdown
Contributor

Summary

scripts/mdm-bootstrap.sh installs Node.js 22.14.0 by default on machines that lack it. Node 22.23.4 fixes the Brotli decompression vulnerability in node:zlib and fetch(), so bump the default. UG_NODE_VERSION overrides are unaffected.

Test plan

  • Run the script on a machine without Node and confirm node --version reports v22.23.4
  • Confirm the tarball URL https://nodejs.org/dist/v22.23.4/node-v22.23.4-<plat>-<arch>.tar.gz resolves

🤖 Generated with Claude Code

Node.js 22.23.4 fixes the Brotli decoder flaw reachable through node:zlib and
fetch(). The bootstrap installs this version on machines without Node, so
the old 22.14.0 default would ship an unpatched runtime.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@greynewell
greynewell marked this pull request as ready for review October 24, 2026 17:33
Copilot AI balanced review requested due to automatic review settings October 24, 2026 17:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The change is narrow, but the supplied evidence does not resolve uncertainty about the release and supported platform tarball availability.

No open issues 🎉

What changed in this PR

Updates the MDM bootstrap script’s default Node.js version to 22.23.4, targeting the Brotli security fix described in the PR.

Changes:

  • Bumps the default from 22.14.0 to 22.23.4.
  • Preserves UG_NODE_VERSION overrides and existing installation logic.
File Description
scripts/​mdm-bootstrap.sh Updates the default Node.js version used during bootstrap.

Review effort: Balanced

@greynewell greynewell mentioned this pull request Oct 24, 2026
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