Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: update outdated OpenSSL 3.x strategy #1211

Merged
merged 2 commits into from
Apr 20, 2022

Conversation

danbev
Copy link
Contributor

@danbev danbev commented Apr 14, 2022

Refs: #1207

@richardlau
Copy link
Member

Refs: #1207

The ask in #1207 was to update the strategy for Node.js 17/18, and in particular the state of continued support for dynamically linking OpenSSL 1.1.1. My own view there is that we could say we support that for as long as OpenSSL 1.1.1 is in support.

Note that OpenSSL 1.1.1 is scheduled to go out of support after 2023-09-11 [1] which is before the End-of-Life dates for Node.js 18, but also Node.js 16 and we have to make a decision at some point regarding what we're going to do with Node.js 16. We have precedent before of shortening the expected lifespan of a Node.js LTS release (Node.js 8) to coincide with the end of support for the OpenSSL version it was built against. If we chose to do that we need to make that decision earlier and communicate that to the wider ecosystem.

(The strategy doc says

OpenSSL 1.1.1 goes EOL on 2023-09-11, which is before 16.x will go EOL, so is not an appropriate choice for 16.x

but OpenSSL 3.0.0 wasn't available when Node.js 16.x went out so we had no choice but to release on OpenSSL 1.1.1.)

We are using the quic fork of OpenSSL 1.1.1 -- I haven't seen any indication that they would differ regarding how long 1.1.1 would be supported for.

[1] https://www.openssl.org/policies/releasestrat.html

@danbev
Copy link
Contributor Author

danbev commented Apr 14, 2022

The ask in #1207 was to update the strategy for Node.js 17/1

This PR intentionally does not address the strategy, that is the reason for using Refs instread of Fixes. Sorry if that was confusing.

This commit also updates the dates for OpenSSL 3.0.0 release in the
OpenSSL versions section.
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@mhdawson mhdawson merged commit 24ea31f into nodejs:main Apr 20, 2022
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.

7 participants