-
Notifications
You must be signed in to change notification settings - Fork 77
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
Adding flag to disable "ask validation" #100
Comments
Made this so you can pass in a cert instead of completely skipping validation |
Much better to provide a cert than to disable validation. Make sure to use token substitution and redaction to ensure that it does not leak through logs or external handlers. |
What is the status of this ticket? |
Hello, Can we please have this option? |
@ltoning it is available in a fork... |
I will be happy to take a look at integrating it once Jeff is ready to create a pull request. Based on the tag it may not be ready for production without further testing. |
@majormoses I am using it for my test and prod environment. Everything is working fine with the |
When running
check-es-node-status
on a cluster with SSL enabled that is using self signed certs, the check will fail with the following:Check failed to run: SSL_connect returned=1 errno=0 state=error: certificate verify failed
The plugin should have a flag to allow skipping the validation of the cert for use cases where you would use self signed certs instead of signed certs.
The text was updated successfully, but these errors were encountered: