Adding d/p/23_sslutils_fix_error_message from upstream
https://patch-diff.githubusercontent.com/raw/monitoring-plugins/monitoring-plugins/pull/2112.patch
This commit is contained in:
parent
45f91e2095
commit
b6a58eb919
2 changed files with 24 additions and 0 deletions
23
debian/patches/23_sslutils_fix_error_message
vendored
Normal file
23
debian/patches/23_sslutils_fix_error_message
vendored
Normal file
|
@ -0,0 +1,23 @@
|
||||||
|
From ef3045b97eea23a4dec3197277e5ff4e5afc5c71 Mon Sep 17 00:00:00 2001
|
||||||
|
From: =?UTF-8?q?Andre=20Kl=C3=A4rner?= <kandre@ak-online.be>
|
||||||
|
Date: Mon, 17 Mar 2025 16:13:46 +0100
|
||||||
|
Subject: [PATCH] change error message for missing certificate
|
||||||
|
|
||||||
|
The old error message is quite similar to the openssl `failed to retrieve issuer certificate` and can mislead users to troubleshooting certificate stores.
|
||||||
|
|
||||||
|
The new message should be distinct enough to make it clear to users that this is not a problem raised by the underlying SSL implementation, but a problem inside monitoring-plugins.
|
||||||
|
---
|
||||||
|
plugins/sslutils.c | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
--- a/plugins/sslutils.c
|
||||||
|
+++ b/plugins/sslutils.c
|
||||||
|
@@ -208,7 +208,7 @@
|
||||||
|
time_t tm_t;
|
||||||
|
|
||||||
|
if (!certificate) {
|
||||||
|
- printf("%s\n",_("CRITICAL - Cannot retrieve server certificate."));
|
||||||
|
+ printf("%s\n",_("CRITICAL - No server certificate present to inspect."));
|
||||||
|
return STATE_CRITICAL;
|
||||||
|
}
|
||||||
|
|
1
debian/patches/series
vendored
1
debian/patches/series
vendored
|
@ -14,4 +14,5 @@
|
||||||
20_check_icmp_fix_rtmin
|
20_check_icmp_fix_rtmin
|
||||||
21_check_http_state_regex
|
21_check_http_state_regex
|
||||||
22_check_by_ssh_missing_options
|
22_check_by_ssh_missing_options
|
||||||
|
23_sslutils_fix_error_message
|
||||||
# feature patches
|
# feature patches
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue