Skip to content

Update Node.js to fix the Brotli security issue (CVE-2026-111497) - #3532

Merged
mattallford merged 1 commit into
mainfrom
ma/node-update
Oct 24, 2026
Merged

mattallford merged 1 commit into
mainfrom
ma/node-update

Conversation

@mattallford

@mattallford mattallford commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Updates Node.js to a fixed release for the Brotli security issue CVE-2026-111497 ("Node2Shell"). This matches the change already made to the other Astro microsites (blog-new, devops, Handbook, integrations).

The fixed Node releases are 22.23.4, 24.22.1 and 26.12.1. Node 20, 23, 25 and earlier won't get a fix.

Changes

  • .github/versions.env: NODE_VERSION goes from 22 to 22.23.4. The PR workflows (branch-markdown.yml, branch-spellcheck.yml) read this value. With only 22, setup-node could pick an older 22.x already cached on the runner.
  • .nvmrc (new): pins 22.23.4, the same version the shared deploy build uses, so local development matches CI.
  • package.json: engines.node goes from >=18.14.1 to ^22.23.4 || ^24.22.1 || >=26.12.1, so vulnerable and end-of-life versions no longer meet the requirement. The pnpm engine is unchanged.

Verification

  • pnpm install on Node 22.23.4: pnpm-lock.yaml is unchanged.
  • pnpm build on Node 22.23.4: passes (3,135 pages).
  • After merge, the push to main starts a deploy build. Its "Setup Node" step should show node: v22.23.4.

@team-marketing-branch-protections

Copy link
Copy Markdown

Pull request environment is available at https://stoctodocspr3532.z22.web.core.windows.net/.

You can view the ephemeral environment status in Octopus Deploy.

This environment will be automatically deprovisioned when the pull request is closed, or after 7 days of inactivity.

@OctoJames OctoJames 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.

looks good to me

@mattallford
mattallford merged commit faad311 into main Oct 24, 2026
8 checks passed
@mattallford
mattallford deleted the ma/node-update branch October 24, 2026 03: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.

3 participants