-
Notifications
You must be signed in to change notification settings - Fork 29.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: export more OpenSSL symbols on Windows #45486
build: export more OpenSSL symbols on Windows #45486
Conversation
Review requested:
|
@mohd-akram The cpp linting is failed, see https://github.com/nodejs/node/actions/runs/3481978405/jobs/5825655808 |
I saw that, but it's for existing lines unrelated to the PR so I didn't change it. I'll fix it? |
That will be good. Looks the task only trigger on changed files. |
Landed in 3abd559 |
@mohd-akram Thx for the contribution :) |
PR-URL: #45486 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
hi @gengjiawen just want to make sure this makes sense as a Could you elaborate a bit more on why do you want to highlight that as a notable change for Node.js end-users? (Also if you do so, it's a good chance to put together a small write-up so that it shows up more than just the commit message in the changelog, similar to the ESM write-up in #46455) 😊 thanks in advance! |
Windows users can use more native functions in openssl like unix user. |
As a side effect, the binary may have become a fair bit bigger on Windows. |
thank you @gengjiawen, unfortunately I had already closed the changelog and ran the release job yesterday in order to promote the builds today. sorry about that! |
Adding the raw commit message in change won't hurt much. But missing it will make lots of people miss it. |
PR-URL: #45486 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
PR-URL: #45486 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
PR-URL: #45486 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Fixes #45445.