Skip to content

Conversation

richardlau
Copy link
Member

Draft as being opened to run/check CI but not intended to merge. Preferred solution to update OpenSSL is to merge #57301 and then run the workflow to perform the OpenSSL update.

First commit is from #57301.
Second commit is the result of running ./tools/dep_updaters/update-openssl.sh download.
Third commit was created by running make gen-openssl -- I had issues running ./tools/dep_updaters/update-openssl.sh regenerate due to missing nasm locally.

@richardlau richardlau added the blocked PRs that are blocked by other issues or PRs. label Mar 3, 2025
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/security-wg

@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 Mar 3, 2025
@nodejs-github-bot
Copy link
Collaborator

@richardlau
Copy link
Member Author

Superseded by #57335.

@richardlau richardlau closed this Mar 5, 2025
@richardlau richardlau deleted the openssl branch March 5, 2025 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked PRs that are blocked by other issues or PRs. 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants