-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
schannel: add CURLOPT_CERTINFO support #822
Conversation
@@ -578,6 +578,7 @@ if(NOT UNIX) | |||
if(HAVE_SCHANNEL_H) | |||
set(USE_SCHANNEL ON) | |||
set(SSL_ENABLED ON) | |||
check_library_exists_concat("crypt32" CertFreeCertificateContext HAVE_LIBCRYPT32) |
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.
Please fix the indentation by adding one additional space in front of this line.
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.
ok
The proposed patch / merge request looks good to me, aside from the small indentation fixes. @bagder do you have any final comments before merging? |
@mback2k hold off there's a patch release pending that's why no features have been added |
@jay Yep, I saw that information. :-) |
my comment: 👍 |
thanks! |
allow to retrieve certinfo from winssl backend