Skip to content

tools: store "default" OpenSSL version in openssl-matrix.nix - #64962

Open
aduh95 wants to merge 1 commit into
nodejs:mainfrom
aduh95:openssl-matrix-default
Open

tools: store "default" OpenSSL version in openssl-matrix.nix#64962
aduh95 wants to merge 1 commit into
nodejs:mainfrom
aduh95:openssl-matrix-default

Conversation

@aduh95

@aduh95 aduh95 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

To help with automating keeping in sync with the bundled version.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Aug 2, 2026
panva
panva previously approved these changes Aug 2, 2026
@aduh95 aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Aug 2, 2026
@panva

panva commented Aug 2, 2026

Copy link
Copy Markdown
Member

Actually, #63949 needs the 3.5 target to stay, its deps don't play nice with 4.0 yet to be able to use that instead.

@panva
panva dismissed their stale review August 2, 2026 16:30

Per above

@aduh95

aduh95 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor Author

Actually, #63949 needs the 3.5 target to stay, its deps don't play nice with 4.0 yet to be able to use that instead.

This PR doesn't change that, the bundled version is OpenSSL 3.5:

MAJOR=3
MINOR=5

# "default" OpenSSL release line, should be kept in sync with the bundled version:
openssl = pkgs.openssl_3_5;

@panva

panva commented Aug 2, 2026

Copy link
Copy Markdown
Member

Thank you.

To help with automating keeping in sync with the bundled version.

Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@aduh95
aduh95 force-pushed the openssl-matrix-default branch from 8296ba0 to 166e844 Compare August 2, 2026 22:31
@github-actions

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants