diff options
author | Barak Shohat <barak@bazzisoft.com> | 2021-05-24 13:42:43 +0300 |
---|---|---|
committer | Barak Shohat <barak@bazzisoft.com> | 2021-05-24 14:53:58 +0300 |
commit | 63cb7ecfcf8d5b6c9f2be704eee7fa7cd9216f88 (patch) | |
tree | 499396380a531826f97c855c592cba2108e697ee | |
parent | 6946b546feb9635413932861c4256f0067846860 (diff) | |
download | monitoring-plugins-63cb7ecfcf8d5b6c9f2be704eee7fa7cd9216f88.tar.gz |
check_curl.c: bugfix: verify certificates option should not force SSL to be used
-rw-r--r-- | plugins/check_curl.c | 2 | ||||
-rw-r--r-- | plugins/t/check_curl.t | 2 |
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" ); }; |