Skip to content

[v22.x] Update to OpenSSL 3.5 - #59859

Closed
richardlau wants to merge 5 commits into
nodejs:v22.x-stagingfrom
richardlau:v22.x-openssl3.5
Closed

[v22.x] Update to OpenSSL 3.5#59859
richardlau wants to merge 5 commits into
nodejs:v22.x-stagingfrom
richardlau:v22.x-openssl3.5

Conversation

@richardlau

@richardlaurichardlau commented Sep 11, 2025

Copy link
Copy Markdown
Member

This is an explicit backport to v22.x-staging of the OpenSSL 3.5 PRs:

These all cherry-pick cleanly, but they are explicitly backported here for visibility.

Updating OpenSSL in Node.js 22.x is necessary for us to continue to support Node.js 22.x through to the planned End-of-Life date of 30 April 2027 as OpenSSL 3.0 goes out of support in September 2026.

The first commit is new and addresses concerns in #59715 by fixing the default security level to 1 to minimize disruption when updating to a newer version of Node.js 22 containing the OpenSSL 3.5 updates.

cc @nodejs/crypto @nodejs/releasers

richardlauand others added 5 commits September 11, 2025 16:35
Node.js 22 was released with OpenSSL 3.0 which had a default security
level of 1. OpenSSL 3.2 bumped this to 2, but we need to fix this at
1 to minimize disruption to users of Node.js 22.x.
PR-URL: nodejs#59234
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
PR-URL: nodejs#59234
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
PR-URL: nodejs#59371
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
PR-URL: nodejs#59371
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@richardlaurichardlau added openssl Issues and PRs related to the OpenSSL dependency. dont-land-on-v20.x v22.x Issues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch. labels Sep 11, 2025
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-botnodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. labels Sep 11, 2025
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@marco-ippolito

Copy link
Copy Markdown
Member

@richardlau can we also backport it to v20?

@nodejs-github-bot

This comment was marked as outdated.

@panvapanva added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label Sep 11, 2025
@nodejs-github-bot

This comment was marked as outdated.

@richardlau

richardlau commented Sep 11, 2025

Copy link
Copy Markdown
MemberAuthor

@richardlau can we also backport it to v20?

I wasn't planning to as Node.js 20 will go End-of-Life before OpenSSL 3.0 does.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

richardlau added a commit that referenced this pull request Sep 16, 2025
Node.js 22 was released with OpenSSL 3.0 which had a default security
level of 1. OpenSSL 3.2 bumped this to 2, but we need to fix this at
1 to minimize disruption to users of Node.js 22.x.
PR-URL: #59859
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
richardlau pushed a commit that referenced this pull request Sep 16, 2025
PR-URL: #59234
Backport-PR-URL: #59859
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
richardlau pushed a commit that referenced this pull request Sep 16, 2025
PR-URL: #59234
Backport-PR-URL: #59859
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
richardlau pushed a commit that referenced this pull request Sep 16, 2025
PR-URL: #59371
Backport-PR-URL: #59859
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
richardlau pushed a commit that referenced this pull request Sep 16, 2025
PR-URL: #59371
Backport-PR-URL: #59859
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@richardlau

Copy link
Copy Markdown
MemberAuthor

Landed in b8870c4...98e399b

richardlau pushed a commit that referenced this pull request Sep 22, 2025
Notable changes:
crypto:
* update root certificates to NSS 3.114 (Node.js GitHub Bot) #59571
deps:
* fix OpenSSL security level at 1 (Richard Lau) #59859
* upgrade openssl sources to openssl-3.5.2 (Node.js GitHub Bot) #59371
doc:
* stabilize --disable-sigusr1 (Rafael Gonzaga) #59707
* mark `path.matchesGlob` as stable (Aviv Keller) #59572
http:
* (SEMVER-MINOR) add Agent.agentKeepAliveTimeoutBuffer option (Haram Jeong) #59315
http2:
* (SEMVER-MINOR) add support for raw header arrays in h2Stream.respond() (Tim Perry) #59455
inspector:
* add http2 tracking support (Darshan Sen) #59611
sea:
* (SEMVER-MINOR) implement execArgvExtension (Joyee Cheung) #59560
* (SEMVER-MINOR) support execArgv in sea config (Joyee Cheung) #59314
stream:
* (SEMVER-MINOR) add brotli support to CompressionStream and DecompressionStream (Matthew Aitken) #59464
test_runner:
* (SEMVER-MINOR) support object property mocking (Idan Goshen) #58438
worker:
* (SEMVER-MINOR) add cpu profile APIs for worker (theanarkh) #59428
PR-URL: #59973
richardlau pushed a commit that referenced this pull request Sep 24, 2025
Notable changes:
crypto:
* update root certificates to NSS 3.114 (Node.js GitHub Bot) #59571
deps:
* fix OpenSSL security level at 1 (Richard Lau) #59859
* upgrade openssl sources to openssl-3.5.2 (Node.js GitHub Bot) #59371
doc:
* stabilize --disable-sigusr1 (Rafael Gonzaga) #59707
* mark `path.matchesGlob` as stable (Aviv Keller) #59572
http:
* (SEMVER-MINOR) add Agent.agentKeepAliveTimeoutBuffer option (Haram Jeong) #59315
http2:
* (SEMVER-MINOR) add support for raw header arrays in h2Stream.respond() (Tim Perry) #59455
inspector:
* add http2 tracking support (Darshan Sen) #59611
sea:
* (SEMVER-MINOR) implement execArgvExtension (Joyee Cheung) #59560
* (SEMVER-MINOR) support execArgv in sea config (Joyee Cheung) #59314
stream:
* (SEMVER-MINOR) add brotli support to CompressionStream and DecompressionStream (Matthew Aitken) #59464
test_runner:
* (SEMVER-MINOR) support object property mocking (Idan Goshen) #58438
worker:
* (SEMVER-MINOR) add cpu profile APIs for worker (theanarkh) #59428
PR-URL: #59973
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-rebaseAdd this label to allow the Commit Queue to land a PR in several commits.dependenciesPull requests that update a dependency file.needs-ciPRs that need a full CI run.opensslIssues and PRs related to the OpenSSL dependency.v22.xIssues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@richardlau@nodejs-github-bot@marco-ippolito@panva@targos@BridgeAR