From: Michael Howe <michael@michaelhowe.org>
Date: Sun, 19 Jun 2011 22:25:04 +0000 (+0000)
Subject: nagios-plugins-local: version 0.5 removes an extraneous print statement in check_cert... 
X-Git-Tag: 0.6~3
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=82161516d0646e882276605a7bf2880a497982d3;p=packages%2Fn%2Fnagios-plugins-local.git

nagios-plugins-local: version 0.5 removes an extraneous print statement in check_cert that was causing nagios to see utter rubbish
---

diff --git a/debian/changelog b/debian/changelog
index 1dc1d0a..0adfda0 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+nagios-plugins-local (0.5) unstable; urgency=low
+
+  * Fixed bug in check_cert plugin printing out certificates when it
+    shouldn't. 
+
+ -- Michael Howe <michael@michaelhowe.org>  Sun, 19 Jun 2011 23:15:25 +0100
+
 nagios-plugins-local (0.4) unstable; urgency=low
 
   * check_cert plugin now handles SMTP TLS. 
diff --git a/plugins/check_cert b/plugins/check_cert
index ba48c54..811bde3 100755
--- a/plugins/check_cert
+++ b/plugins/check_cert
@@ -205,7 +205,6 @@ exit($ERRORS{'OK'});
 sub s_client {
     my ($command, $arguments) = @_;
 
-    print scalar(`/bin/echo $command | /usr/bin/gnutls-cli --insecure --print-cert $arguments 2>/dev/null | /usr/bin/openssl x509 -text 2>/dev/null`);
     return(parse_certificate(scalar(`/bin/echo $command | /usr/bin/gnutls-cli --insecure --print-cert $arguments 2>/dev/null | /usr/bin/openssl x509 -text 2>/dev/null`)));
 }