Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 27f183a

Browse filesBrowse files
richardlauaduh95
authored andcommitted
doc: update maintaining-openssl.md for openssl
Update the instructions for maintaining OpenSSL in the Node.js source tree to reflect switching back from the quictls fork of OpenSSL back to official OpenSSL. PR-URL: #57413 Refs: #57301 Refs: #57142 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>
1 parent 5be80b1 commit 27f183a
Copy full SHA for 27f183a

File tree

Expand file treeCollapse file tree

2 files changed

+11
-36
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

2 files changed

+11
-36
lines changed
Open diff view settings
Collapse file

‎.github/workflows/update-openssl.yml‎

Copy file name to clipboardExpand all lines: .github/workflows/update-openssl.yml
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ jobs:
3535
author: Node.js GitHub Bot <github-bot@iojs.org>
3636
body: This is an automated update of OpenSSL to ${{ env.NEW_VERSION }}.
3737
branch: actions/tools-update-openssl # Custom branch *just* for this Action.
38-
commit-message: 'deps: upgrade openssl sources to quictls/openssl-${{ env.NEW_VERSION }}'
38+
commit-message: 'deps: upgrade openssl sources to openssl-${{ env.NEW_VERSION }}'
3939
labels: dependencies, openssl
4040
title: 'deps: update OpenSSL to ${{ env.NEW_VERSION }}'
4141
path: deps/openssl
Collapse file

‎doc/contributing/maintaining/maintaining-openssl.md‎

Copy file name to clipboardExpand all lines: doc/contributing/maintaining/maintaining-openssl.md
+10-35Lines changed: 10 additions & 35 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -4,28 +4,6 @@ OpenSSL is automatically updated by the [update-openssl-action][].
44
There is also a script in `tools/dep_updaters` that can be used to update it.
55
This document describes how to manually update `deps/openssl/`.
66

7-
If you need to provide updates across all active release lines you will
8-
currently need to generate four PRs as follows:
9-
10-
* a PR for `main` which is generated following the instructions
11-
below for OpenSSL 3.x.x.
12-
* a PR for 18.x following the instructions in the v18.x-staging version
13-
of this guide.
14-
* a PR for 16.x following the instructions in the v16.x-staging version
15-
of this guide.
16-
17-
## Use of the quictls/openssl fork
18-
19-
Node.js currently uses the quictls/openssl fork, which closely tracks
20-
the main openssl/openssl releases with the addition of APIs to support
21-
the QUIC protocol.
22-
23-
Details on the fork, as well as the latest sources, can be found at
24-
<https://github.com/quictls/openssl>.
25-
26-
Branches are used per OpenSSL version (for instance,
27-
<https://github.com/quictls/openssl/tree/OpenSSL_1_1_1j+quic>).
28-
297
## Requirements
308

319
* Linux environment.
@@ -52,35 +30,33 @@ NASM version 2.11.08
5230

5331
## 1. Obtain and extract new OpenSSL sources
5432

55-
Get a new source from <https://github.com/quictls/openssl/tree/openssl-3.0.5+quic>
33+
Get a new source from <https://github.com/openssl/openssl/tree/openssl-3.0.16>
5634
and copy all files into `deps/openssl/openssl`. Then add all files and commit
5735
them. (The link above, and the branch, will change with each new OpenSSL
5836
release).
5937

60-
### OpenSSL 3.x.x
61-
6238
```bash
63-
git clone https://github.com/quictls/openssl
39+
git clone https://github.com/openssl/openssl
6440
cd openssl
6541
cd ../node/deps/openssl
6642
rm -rf openssl
6743
cp -R ../../../openssl openssl
68-
rm -rf openssl/.git* openssl/.travis*
44+
rm -rf openssl/.git*
6945
git add --all openssl
7046
git commit openssl
7147
```
7248

7349
```text
74-
deps: upgrade openssl sources to quictls/openssl-3.0.5+quic
50+
deps: upgrade openssl sources to openssl-3.0.16
7551
7652
This updates all sources in deps/openssl/openssl by:
77-
$ git clone git@github.com:quictls/openssl.git
53+
$ git clone git@github.com:openssl/openssl.git
7854
$ cd openssl
79-
$ git checkout openssl-3.0.5+quic
55+
$ git checkout openssl-3.0.16
8056
$ cd ../node/deps/openssl
8157
$ rm -rf openssl
8258
$ cp -R ../../../openssl openssl
83-
$ rm -rf openssl/.git* openssl/.travis*
59+
$ rm -rf openssl/.git*
8460
$ git add --all openssl
8561
$ git commit openssl
8662
```
@@ -99,7 +75,8 @@ make -C deps/openssl/config clean
9975
make -C deps/openssl/config
10076
```
10177

102-
**Note**: If the 32-bit Windows is failing to compile run this workflow instead:
78+
Fix up 32-bit Windows assembler directives. This will allow the commits to be
79+
cherry-picked to older release lines that still provide binaries on 32-bit Windows.
10380

10481
```bash
10582
make -C deps/openssl/config clean
@@ -140,10 +117,8 @@ git commit
140117
The commit message can be written as (with the openssl version set
141118
to the relevant value):
142119

143-
### OpenSSL 3.x.x
144-
145120
```text
146-
deps: update archs files for quictls/openssl-3.0.5+quic
121+
deps: update archs files for openssl-3.0.16
147122
148123
After an OpenSSL source update, all the config files need to be
149124
regenerated and committed by:

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.