Skip to content
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 #1897) : skip server SSL certs verification when configured #2174

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dolifer
Copy link

@dolifer dolifer commented Jan 17, 2024

An attempt to fix #1897

I've updated RestService to respect theenableSslCertificateVerification option, even when there are no client certificates provided.

This ensures that in case of self-signed certificate, we can get schema from schema-registry, by skiping certificate validation.

Copy link

cla-assistant bot commented Jan 17, 2024

CLA assistant check
All committers have signed the CLA.

@dolifer
Copy link
Author

dolifer commented Jan 19, 2024

@mhowlett hello, here is my attempt to fix the issue with certificates validation.

@jorgebg90
Copy link

Hello,
Is there any update on this? @dolifer your solution jumps over the self-signed certificate or uses the specified one with the property SslCaLocation?

Thanks in advance :)

@dolifer
Copy link
Author

dolifer commented Apr 19, 2024

@jorgebg90 it's jumps over, and skips the validation at all by using custom ServerCertificateCustomValidationCallback which "trusts" ANY given certificate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants