Faidon Liambotis has submitted this change and it was merged.
Change subject: nagios: fix check_ssl warning when no SANs are present
......................................................................
nagios: fix check_ssl warning when no SANs are present
Change-Id: I4b49486c19014098b13dc00c3831947ee950dc21
---
M modules/nagios_common/files/check_commands/check_ssl
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Filippo Giunchedi: Looks good to me, but someone else must approve
Faidon Liambotis: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/nagios_common/files/check_commands/check_ssl
b/modules/nagios_common/files/check_commands/check_ssl
index 5812f82..ea27365 100755
--- a/modules/nagios_common/files/check_commands/check_ssl
+++ b/modules/nagios_common/files/check_commands/check_ssl
@@ -182,7 +182,7 @@
push @verbose, "Certificate subject: $subject";
push @verbose, "Certificate issuer: $issuer";
- push @verbose, "Certificate SANs: $sans";
+ push @verbose, "Certificate SANs: $sans" if defined $sans;
if ( $ng->subject && $subject !~ /\Q$ng->{subject}\E/ ) {
push @crit, "Certificate subject $subject didn't match argument";
--
To view, visit https://gerrit.wikimedia.org/r/301641
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4b49486c19014098b13dc00c3831947ee950dc21
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
