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
Fix no-tls1_2 and no-dtls1_2 #2788
Conversation
ADD_TEST(test_early_cb); | ||
#endif |
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.
So... this works with SSL v3, TLS v1.0, v1.1 and v1.3, but not v1.2?
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.
No - this test only works if we have v1.2, i.e. if 1.2 is disabled we shouldn't run it. It uses some 1.2 only ciphersuites.
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.
I read the directive wrong, sorry. Thanks for explaining.
Pushed. Thanks. |
Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from #2788)
Reviewed-by: Richard Levitte <levitte@openssl.org> (Merged from #2788)
Checklist
Description of change
This fixes no-tls1_2 and no-dtls1_2 configurations. The commit changing sslapitest is for master only. The other commit is for both master and 1.1.0.