-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
ssl: Fix cert_auth check #8367
ssl: Fix cert_auth check #8367
Conversation
CT Test Results 2 files 66 suites 47m 6s ⏱️ Results for commit a2c9913. ♻️ This comment has been updated with latest results. To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass. See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally. Artifacts// Erlang/OTP Github Action Bot |
a0c0530
to
59489fc
Compare
59489fc
to
0823551
Compare
lib/ssl/test/ssl_cert_SUITE.erl
Outdated
@@ -981,6 +984,25 @@ key_auth_ext_sign_only(Config) when is_list(Config) -> | |||
ssl_test_lib:basic_test(ClientOpts, ServerOpts, Config). | |||
|
|||
%%-------------------------------------------------------------------- | |||
cert_auth_in_first_ca() -> | |||
[{doc,"Test cert auth will be available in firs ca in chain, make it happen by only having one"}]. |
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.
@IngelaAndin, thank you for fixing this for us. A minor comment about a typo in the test description for "first".
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.
Thanks for noticing, fixed.
Include also end entity cert when selecting issuers for cert_auth extension check Closes erlang#8356
0823551
to
a2c9913
Compare
Include also end entity cert when selecting issuers for cert_auth extension check
Closes #8356