aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Barak Shohat <barak@bazzisoft.com> 2021-05-24 13:42:43 +0300
committerGravatar Barak Shohat <barak@bazzisoft.com> 2021-05-24 14:53:58 +0300
commit63cb7ecfcf8d5b6c9f2be704eee7fa7cd9216f88 (patch)
tree499396380a531826f97c855c592cba2108e697ee
parent6946b546feb9635413932861c4256f0067846860 (diff)
downloadmonitoring-plugins-63cb7ecfcf8d5b6c9f2be704eee7fa7cd9216f88.tar.gz
check_curl.c: bugfix: verify certificates option should not force SSL to be used
-rw-r--r--plugins/check_curl.c2
-rw-r--r--plugins/t/check_curl.t2
2 files changed, 2 insertions, 2 deletions
diff --git a/plugins/check_curl.c b/plugins/check_curl.c
index 3e0a6f94..d29db0a6 100644
--- a/plugins/check_curl.c
+++ b/plugins/check_curl.c
@@ -1346,7 +1346,7 @@ process_arguments (int argc, char **argv)
#ifdef LIBCURL_FEATURE_SSL
case 'D': /* verify peer certificate & host */
verify_peer_and_host = TRUE;
- goto enable_ssl;
+ break;
#endif
case 'S': /* use SSL */
#ifdef LIBCURL_FEATURE_SSL
diff --git a/plugins/t/check_curl.t b/plugins/t/check_curl.t
index 45ee5339..ada6a045 100644
--- a/plugins/t/check_curl.t
+++ b/plugins/t/check_curl.t
@@ -95,7 +95,7 @@ SKIP: {
$res = NPTest->testCmd("./$plugin -v -H $host_tls_http:443 -S -p 443");
like( $res->output, '/^Host: '.$host_tls_http.'\s*$/ms', "Host Header OK" );
- $res = NPTest->testCmd("./$plugin -v -H $host_tls_http -D -p 443");
+ $res = NPTest->testCmd("./$plugin -v -H $host_tls_http -D -S -p 443");
like( $res->output, '/(^Host: '.$host_tls_http.'\s*$)|(cURL returned 60)/ms', "Host Header OK" );
};