-
Notifications
You must be signed in to change notification settings - Fork 29.6k
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
(v8.x backport) crypto: reuse variable instead of reevaluation #19114
Conversation
PR-URL: nodejs#17735 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Jon Moss <me@jonathanmoss.me> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
potentially related failure in debian 8 https://ci.nodejs.org/job/node-test-commit-linux/17205/nodes=debian8-x86/console
|
This PR should not affect TLS tests. I started two more CIs, the first failed |
Thanks for the heads up landed in b43a59c |
Backport of #17735 to v8.x.