Skip to content

deps: update OpenSSL 3.0.4+quic - #43536

Closed
RafaelGSS wants to merge 2 commits into
nodejs:mainfrom
RafaelGSS:update-openssl-3-0-4
Closed

deps: update OpenSSL 3.0.4+quic#43536
RafaelGSS wants to merge 2 commits into
nodejs:mainfrom
RafaelGSS:update-openssl-3-0-4

Conversation

@RafaelGSS

Copy link
Copy Markdown
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
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp

@nodejs-github-botnodejs-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
@aduh95aduh95 changed the title deps: update OpenSSL 1.1.1p+quicdeps: update OpenSSL 3.0.4p+quicJun 22, 2022
@richardlaurichardlau changed the title deps: update OpenSSL 3.0.4p+quicdeps: update OpenSSL 3.0.4+quicJun 22, 2022
@richardlaurichardlau added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 22, 2022
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 22, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@richardlaurichardlau 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
Copy Markdown
Member

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

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@RafaelGSS

Copy link
Copy Markdown
MemberAuthor

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

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@richardlau

Copy link
Copy Markdown
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
Copy Markdown
MemberAuthor

Recreating from scratch.

@RafaelGSS

Copy link
Copy Markdown
MemberAuthor

#43589

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@RafaelGSS@nodejs-github-bot@tniessen@richardlau