summaryrefslogtreecommitdiffstats
path: root/plugins
AgeCommit message (Collapse)AuthorFilesLines
2021-10-02Revert to poor man's logicrefs/pull/1707/headRincewindsHat3-18/+16
2021-10-02Change all to comments to old comment styleRincewindsHat1-3/+4
2021-09-25Fix comparing logicRincewindsHat1-2/+2
2021-09-25Remove spaces from testsRincewindsHat1-3/+3
2021-09-25check_swap: Fix perfdata und thresholds for big values and simplify coderincewind1-102/+137
The original problem was https://github.com/monitoring-plugins/monitoring-plugins/pull/1705 where the performance data output of check_swap did not conform to the parser logic of a monitoring system (which decided to go for "correct" SI or IEC units. The PR was accompanied by a change to byte values in the performance data which broke the _perfdata_ helper function which could not handle values of this size. The fix for this, was to use _fperfdata_ which could, but would use float values. I didn't like that (since all values here are discreet) and this is my proposal for a fix for the problem. It introduces some helper functions which do now explicitely work with (u)int64_t, including a special version of the _perfdata_ helper. In the process of introducing this to check_swap, I stumbled over several sections of the check_swap code which I found problematic. Therefore I tried to simplify the code and make it more readable and less redundant. I am kinda sorry about this, but sincerely hope my changes can be helpful.
2021-09-25Introduce new perfdata functions and stuff for using (u)int64_trincewind2-0/+130
2021-09-02Improvements suggested by tobiaswieseJonny007-MKD2-3/+3
2021-09-02check_dns: Fix memory leakJonny007-MKD1-1/+1
2021-09-02check_dns: add --expect-nxdomainJonny007-MKD2-18/+55
2021-08-24Merge pull request #1388 from waja/check_proc_consumption_1357Sven Nierlein1-4/+1
check_disk: Fixing the stuff that is broken on btrfs
2021-07-28Merge branch 'master' into feature_check_curlrefs/pull/1700/headAndreas Baumann2-4/+2
2021-07-25check_curl: fixed a potential buffer overflow in retir/uri_stringAndreas Baumann1-2/+2
2021-07-25check_curl: fixed DNS caching for SSL hostnames (avoid CURLOPT_RESOLVE entry ↵Andreas Baumann1-3/+51
errors)
2021-07-08Merge pull request #1690 from ↵Sven Nierlein1-2/+2
xFuture603/change_state_from_infinite_redirect_loop check_http: changed 'STATE_CRITICAL' to 'STATE_WARNING' for infinite loop
2021-07-04check_curl: changed to STATE_CRITICAL for infinite loops (-ffollow)Andreas Baumann1-2/+2
2021-06-25Let ssh decide if a host is valid, enables usage of ssh .config filerefs/pull/1691/headLorenz Kästle1-2/+0
2021-06-24fix for missing X509_up_ref on old systems with only OpenSSL 1.0Andreas Baumann1-0/+6
2021-06-24changed 'STATE_CRITICAL' to 'STATE_WARNING' for infinite looprefs/pull/1690/headDaniel Uhlmann1-2/+2
2021-06-24added option --enable-automatic-decompressionAndreas Baumann1-1/+16
2021-05-27check_curl:refs/pull/1689/headAndreas Baumann1-0/+12
- added verbose output in verify_callback - pin refcounting for certs (avoid subject extraction error when checking certs in is_openssl_callback mode)
2021-05-20tests: ignore sigpipes in https test daemonrefs/pull/1686/headSven Nierlein1-4/+8
check_http closes the connection after checking the certificate with -C. This leads to sigpipe errors when the ssl daemon wants to send a response and the daemon quits which makes the subsequent tests fail.
2021-05-20tests: increase startup sleepSven Nierlein2-4/+6
github action might fail from to time otherwise
2021-05-19Refactor check_fpingJacob Hansen1-15/+14
* Set correct amount of tests based on conditionals. * When running the test as non-root, we would previously check is the setuid bit is set. This doesn't seem to be needed, so just check if the binary is executable for the user running the test. * Use cmp_ok to check if tests succeeds rather than couting. Signed-off-by: Jacob Hansen <jhansen@op5.com>
2021-05-18check_curl.t: Adjust test to use -f curlJacob Hansen1-2/+1
It appears that `-f follow` doesn't work correctly in `check_curl` at the moment. Test adjusted to use `-f curl` instead. Issue for the above created: https://github.com/monitoring-plugins/monitoring-plugins/issues/1685 Signed-off-by: Jacob Hansen <jhansen@op5.com>
2021-04-30Fix conditional tests for check_http, check_curlAksel Sjögren2-4/+4
Set correct number of tests in skip- blocks to avoid the error "Bad plan. You planned 50 tests but ran 55" when run with/without /usr/bin/faketime and NP_INTERNET_ACCESS=yes/no.
2021-04-30migrate travis ci to github actionsSven Nierlein1-54/+0
2021-04-30fix fping testSven Nierlein1-6/+3
in case of dns errors, fping returns an unknown and thats ok.
2021-04-13check_curl: Fix bug where headers beginning with HTTP_ cause the status line ↵Barak Shohat1-1/+1
parsing to fail.
2021-04-08check_curl: backported --show-body/-B to print body (from check_http)refs/pull/1671/headAndreas Baumann1-4/+15
2021-04-08check_curl: make -C obvious (from check_http, part 2)Andreas Baumann1-0/+4
2021-04-08check_curl: make -C obvious (from check_http)Andreas Baumann1-1/+3
2021-04-08check_curl: Increase regexp limit (to 1024 as in check_http)Andreas Baumann1-1/+1
2021-04-07Merge pull request #1669 from bazzisoft/verify-certSven Nierlein2-4/+18
Add an option to check_curl to verify the peer certificate & host using the system CA's
2021-04-07Merge pull request #1554 from stblassitude/masterSven Nierlein1-1/+7
Docs check_http: make -C obvious
2021-04-07Merge pull request #1566 from hydrapolic/masterSven Nierlein1-1/+1
plugins: check_http: Increase regexp limit
2021-04-07Merge pull request #1465 from lausser/masterSven Nierlein1-2/+3
check_snmp: put the "c" (to mark a counter) after the perfdata value
2021-04-07Merge pull request #1663 from waja/debian_982847_2waja1-1/+1
Using snprintf which honors the buffers size and guarantees null temination. (Closes: #1601)
2021-04-07Add an option to check_curl to verify the peer certificate & host using the ↵refs/pull/1669/headBarak Shohat2-4/+18
system CA's
2021-04-07Merge pull request #1668 from bazzisoft/masterSven Nierlein2-5/+5
Update check_curl.c to display a specific human-readable error message where possible
2021-04-07Updated check_curl.c to display a specific human-readable error message ↵refs/pull/1668/headBarak Shohat2-5/+5
where possible
2021-04-07Merge pull request #1666 from sni/check_curl_fix_header_parserSven Nierlein2-4/+14
fix check_curl crash if http header contains leading spaces
2021-04-07Merge pull request #1644 from ghciv6/check_mysql_query_1562Sven Nierlein1-6/+6
- fix for issue #1562
2021-03-11fix check_curl crash if http header contains leading spacesrefs/pull/1666/headSven Nierlein2-4/+14
check_curl crashes when a (broken) http server returns invalid http header with leading spaces or double colons. This PR adds a fix and a test case for this. Signed-off-by: Sven Nierlein <sven@nierlein.de>
2021-03-08check_curl: added string_statuscode function for printing HTTP/1.1 and ↵refs/pull/1667/headAndreas Baumann1-5/+28
HTTP/2 correctly
2021-02-15Using snprintf which honors the buffers size and guarantees null ↵refs/pull/1663/headFlorian Lohoff1-1/+1
termination. (Closes: #1601) As strcpy may overflow the resulting buffer: flo@p5:~$ /tmp/f/usr/lib/nagios/plugins/check_pgsql -d "$(seq 1 10000)" *** buffer overflow detected ***: terminated Aborted I would propose to change the code rather like this, using snprintf which honors the buffers size and guarantees null termination.
2021-02-08Merge branch 'master' into feature_check_curlAndreas Baumann1-3/+17
2021-01-19check_curl: fixed a potential buffer overflow in url bufferAndreas Baumann1-1/+1
2021-01-19check_curl: fixed help, usage and errors for TLS 1.3Andreas Baumann1-2/+2
2020-11-02check_dns: split multiple IP addresses passed in one -a argumentrefs/pull/1649/headRolf Eike Beer1-3/+17
2020-09-23Merge pull request #1514 from Rasp8e/masterSven Nierlein1-0/+15
Proxy authorization is now kept when using PROXY + CONNECT to HTTPS