Skip to content

Conversation

@RafaelGSS
Copy link
Member

Updated openssl dep to openssl-3.0.4p+quic using the maintenance guide.

Refs: https://mta.openssl.org/pipermail/openssl-announce/2022-June/000228.html

This updates all sources in deps/openssl/openssl by: $ git clone git@github.com:quictls/openssl.git $ cd openssl $ git checkout openssl-3.0.4+quic $ cd ../node/deps/openssl $ rm -rf openssl $ cp -R ../../../openssl openssl $ rm -rf openssl/.git* openssl/.travis* $ git add --all openssl $ git commit openssl
After an OpenSSL source update, all the config files need to be regenerated and committed by: $ make -C deps/openssl/config $ git add deps/openssl/config/archs $ git add deps/openssl/openssl $ git commit
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/gyp
@nodejs-github-bot nodejs-github-bot added dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. openssl Issues and PRs related to the OpenSSL dependency. labels Jun 22, 2022
@aduh95 aduh95 changed the title deps: update OpenSSL 1.1.1p+quic deps: update OpenSSL 3.0.4p+quic Jun 22, 2022
@richardlau richardlau changed the title deps: update OpenSSL 3.0.4p+quic deps: update OpenSSL 3.0.4+quic Jun 22, 2022
@richardlau richardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 22, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 22, 2022
@richardlau richardlau added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label Jun 22, 2022
@tniessen
Copy link
Member

The diff size always bothers me. Hundreds of thousands of lines...

@RafaelGSS
Copy link
Member Author

@richardlau it keeps failing, and I haven't found any insights in the logs. Do you have any idea?

@richardlau
Copy link
Member

@richardlau it keeps failing, and I haven't found any insights in the logs. Do you have any idea?

Hmm I've not seen this before -- the PR rebasing onto the base branch (main) somehow isn't working 😞. I'm not sure why.

@RafaelGSS
Copy link
Member Author

Recreating from scratch.

@RafaelGSS RafaelGSS closed this Jun 27, 2022
@RafaelGSS
Copy link
Member Author

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

Labels

commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. dependencies Pull requests that update a dependency file. needs-ci PRs that need a full CI run. openssl Issues and PRs related to the OpenSSL dependency.

4 participants