Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
2021-09-25 | check_swap: Fix perfdata und thresholds for big values and simplify code | rincewind | 1 | -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-25 | Introduce new perfdata functions and stuff for using (u)int64_t | rincewind | 2 | -0/+130 | |
2021-09-02 | Improvements suggested by tobiaswiese | Jonny007-MKD | 2 | -3/+3 | |
2021-09-02 | check_dns: Fix memory leak | Jonny007-MKD | 1 | -1/+1 | |
2021-09-02 | check_dns: add --expect-nxdomain | Jonny007-MKD | 2 | -18/+55 | |
2021-08-24 | Merge pull request #1388 from waja/check_proc_consumption_1357 | Sven Nierlein | 1 | -4/+1 | |
check_disk: Fixing the stuff that is broken on btrfs | |||||
2021-07-28 | Merge branch 'master' into feature_check_curlrefs/pull/1700/head | Andreas Baumann | 2 | -4/+2 | |
2021-07-25 | check_curl: fixed a potential buffer overflow in retir/uri_string | Andreas Baumann | 1 | -2/+2 | |
2021-07-25 | check_curl: fixed DNS caching for SSL hostnames (avoid CURLOPT_RESOLVE entry ↵ | Andreas Baumann | 1 | -3/+51 | |
errors) | |||||
2021-07-08 | Merge pull request #1690 from ↵ | Sven Nierlein | 1 | -2/+2 | |
xFuture603/change_state_from_infinite_redirect_loop check_http: changed 'STATE_CRITICAL' to 'STATE_WARNING' for infinite loop | |||||
2021-07-04 | check_curl: changed to STATE_CRITICAL for infinite loops (-ffollow) | Andreas Baumann | 1 | -2/+2 | |
2021-06-25 | Let ssh decide if a host is valid, enables usage of ssh .config filerefs/pull/1691/head | Lorenz Kästle | 1 | -2/+0 | |
2021-06-24 | fix for missing X509_up_ref on old systems with only OpenSSL 1.0 | Andreas Baumann | 1 | -0/+6 | |
2021-06-24 | changed 'STATE_CRITICAL' to 'STATE_WARNING' for infinite looprefs/pull/1690/head | Daniel Uhlmann | 1 | -2/+2 | |
2021-06-24 | added option --enable-automatic-decompression | Andreas Baumann | 1 | -1/+16 | |
2021-05-27 | check_curl:refs/pull/1689/head | Andreas Baumann | 1 | -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-24 | check_curl.c: bugfix: verify certificates option should not force SSL to be usedrefs/pull/1688/head | Barak Shohat | 2 | -2/+2 | |
2021-05-20 | tests: ignore sigpipes in https test daemonrefs/pull/1686/head | Sven Nierlein | 1 | -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-20 | tests: increase startup sleep | Sven Nierlein | 2 | -4/+6 | |
github action might fail from to time otherwise | |||||
2021-05-19 | Refactor check_fping | Jacob Hansen | 1 | -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-18 | check_curl.t: Adjust test to use -f curl | Jacob Hansen | 1 | -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-30 | Fix conditional tests for check_http, check_curl | Aksel Sjögren | 2 | -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-30 | migrate travis ci to github actions | Sven Nierlein | 1 | -54/+0 | |
2021-04-30 | fix fping test | Sven Nierlein | 1 | -6/+3 | |
in case of dns errors, fping returns an unknown and thats ok. | |||||
2021-04-13 | check_curl: Fix bug where headers beginning with HTTP_ cause the status line ↵ | Barak Shohat | 1 | -1/+1 | |
parsing to fail. | |||||
2021-04-08 | check_curl: backported --show-body/-B to print body (from check_http)refs/pull/1671/head | Andreas Baumann | 1 | -4/+15 | |
2021-04-08 | check_curl: make -C obvious (from check_http, part 2) | Andreas Baumann | 1 | -0/+4 | |
2021-04-08 | check_curl: make -C obvious (from check_http) | Andreas Baumann | 1 | -1/+3 | |
2021-04-08 | check_curl: Increase regexp limit (to 1024 as in check_http) | Andreas Baumann | 1 | -1/+1 | |
2021-04-07 | Merge pull request #1669 from bazzisoft/verify-cert | Sven Nierlein | 2 | -4/+18 | |
Add an option to check_curl to verify the peer certificate & host using the system CA's | |||||
2021-04-07 | Merge pull request #1554 from stblassitude/master | Sven Nierlein | 1 | -1/+7 | |
Docs check_http: make -C obvious | |||||
2021-04-07 | Merge pull request #1566 from hydrapolic/master | Sven Nierlein | 1 | -1/+1 | |
plugins: check_http: Increase regexp limit | |||||
2021-04-07 | Merge pull request #1465 from lausser/master | Sven Nierlein | 1 | -2/+3 | |
check_snmp: put the "c" (to mark a counter) after the perfdata value | |||||
2021-04-07 | Merge pull request #1663 from waja/debian_982847_2 | waja | 1 | -1/+1 | |
Using snprintf which honors the buffers size and guarantees null temination. (Closes: #1601) | |||||
2021-04-07 | Add an option to check_curl to verify the peer certificate & host using the ↵refs/pull/1669/head | Barak Shohat | 2 | -4/+18 | |
system CA's | |||||
2021-04-07 | Merge pull request #1668 from bazzisoft/master | Sven Nierlein | 2 | -5/+5 | |
Update check_curl.c to display a specific human-readable error message where possible | |||||
2021-04-07 | Updated check_curl.c to display a specific human-readable error message ↵refs/pull/1668/head | Barak Shohat | 2 | -5/+5 | |
where possible | |||||
2021-04-07 | Merge pull request #1666 from sni/check_curl_fix_header_parser | Sven Nierlein | 2 | -4/+14 | |
fix check_curl crash if http header contains leading spaces | |||||
2021-04-07 | Merge pull request #1644 from ghciv6/check_mysql_query_1562 | Sven Nierlein | 1 | -6/+6 | |
- fix for issue #1562 | |||||
2021-03-11 | fix check_curl crash if http header contains leading spacesrefs/pull/1666/head | Sven Nierlein | 2 | -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-08 | check_curl: added string_statuscode function for printing HTTP/1.1 and ↵refs/pull/1667/head | Andreas Baumann | 1 | -5/+28 | |
HTTP/2 correctly | |||||
2021-02-15 | Using snprintf which honors the buffers size and guarantees null ↵refs/pull/1663/head | Florian Lohoff | 1 | -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-08 | Merge branch 'master' into feature_check_curl | Andreas Baumann | 1 | -3/+17 | |
2021-01-19 | check_curl: fixed a potential buffer overflow in url buffer | Andreas Baumann | 1 | -1/+1 | |
2021-01-19 | check_curl: fixed help, usage and errors for TLS 1.3 | Andreas Baumann | 1 | -2/+2 | |
2021-01-03 | check_procs -- update test data toorefs/pull/1654/head | Wolfgang Karall-Ahlborn | 1 | -1/+1 | |
2021-01-03 | check_procs -- exchange needle and haystack in strstr() for proper state match | Wolfgang Karall-Ahlborn | 1 | -1/+1 | |
2020-11-02 | check_dns: split multiple IP addresses passed in one -a argumentrefs/pull/1649/head | Rolf Eike Beer | 1 | -3/+17 | |
2020-09-23 | Merge pull request #1514 from Rasp8e/master | Sven Nierlein | 1 | -0/+15 | |
Proxy authorization is now kept when using PROXY + CONNECT to HTTPS | |||||
2020-09-15 | - fix for issue #1562refs/pull/1644/head | ghciv6 | 1 | -6/+6 | |
1st compare then free up resources |