-
Notifications
You must be signed in to change notification settings - Fork 47
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
check-ssl-qualys.rb regularly times out #35
Comments
The check of this host is flapping, passing about once every 4 days or so. The host is a little unusual in that it HTTP 301s to another domain. I would not expect that to have any effect on a TLS check but it is the only notable difference between this host and the others that I'm monitoring. I haven't tried to determine if this is due to check's behavior or something goofy going on with ssllabs.
|
It should be fine, I was testing with google.com last night:
I am working on a fix for this. |
Btw after playing with it more I was able to replicate similar times when using the Btw I also noticed this which confirms that we expect it to be slow: https://github.com/sensu-plugins/sensu-plugins-ssl/blob/1.5.0/bin/check-ssl-qualys.rb#L30-L33 I am working on revamping the code |
OK I have a working version, will create a PR shortly. |
As discovered and troubleshot in #34 the api takes a while to respond, we should wait for the ETA when possible rather than some arbitrary number. This will also indirectly help better handle rate limit issues although handling it properly (429) is out of scope here.
The text was updated successfully, but these errors were encountered: