-
Notifications
You must be signed in to change notification settings - Fork 30k
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
test: update OpenSSL 3.x expected error message #38164
Conversation
This started failing with OpenSSL 3 alpha 14 (nodejs/build#2613 (comment)). 👍 to fast track to unbreak the CI. |
The failure in node-test-linux-linked-openssl300 is expected and should be fixed by #38165. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
That PR has landed so now we should be able to get a green CI.... |
This commit updates an expected error message which has changed in OpenSSl 3.x. OpenSSL 3.0 is still under development but once there is a relase we should not be seeing test failure like this. PR-URL: nodejs#38164 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Michael Dawson <midawson@redhat.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
d198e20
to
7216eb6
Compare
Landed in 7216eb6 |
This commit updates an expected error message which has changed in
OpenSSl 3.x. OpenSSL 3.0 is still under development but once there is a
relase we should not be seeing test failure like this.