[Nagiosplug-devel] [nagios-plugins] fixed small bug in in previous patch (#11)

[William Leibzon]

my previous certificate expiration check patch had a bug and would not have properly checked if warning threshold had numeric values, the actual threshold checking code had no problems

You can merge this Pull Request by running:

  git pull https://github.com/willixix/nagios-plugins master

Or you can view, comment on it, or merge it online at:

  https://github.com/nagios-plugins/nagios-plugins/pull/11

-- Commit Summary --

* applied patch that adds both critical and warning thresholds to certificate expiration checks of check_tcp, check_http, check_smtp
* manually merged latest update that added SSL version parameter into my code that adds warn and crit thresholds for SSL
* added test case for certificate checs with both warning and critical values
* fixing bug in certificate experiation patch checking for bad warning threshold data
* merging my bug fixes with master branch

-- File Changes --

M plugins/check_http.c (6)
M plugins/check_smtp.c (6)
M plugins/check_tcp.c (2)

-- Patch Links --

  https://github.com/nagios-plugins/nagios-plugins/pull/11.patch
  https://github.com/nagios-plugins/nagios-plugins/pull/11.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/nagios-plugins/nagios-plugins/pull/11