summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2021-10-26Reformat a little bit for easier readingRincewindsHat1-16/+32
2021-10-26Recognice IEC binary units and format accordingly to base2 (always)RincewindsHat1-12/+15
2021-10-25Update check_load.tHarald Jenny1-2/+2
adjust tests for new plugin output
2021-10-25Create check_load.cHarald Jenny1-1/+1
just a cosmetic fix so the load plugin display a LOAD prefix before check results
2021-10-25actions: map utmp from host to containerSven Nierlein1-0/+1
this fixes the check_users not having any logged in user to check.
2021-10-20Add comment to make the purpose of the nickname fix more obviousLorenz Kästle1-0/+2
2021-10-20Restrict the nickname length of the test user for check_ircdLorenz Kästle1-1/+1
check_ircd was using the string `ircd` plus the PID as a nickname for connecting to a IRC network by default. This caused errors, when the PID was too high and the network restricted the length of the nickname to 9 characters. This patch "fixes" this by just cutting it of, if it gets too big.
2021-10-03Merge remote-tracking branch 'monitoring-plugins/pr/1707'Holger Weiss4-104/+268
* monitoring-plugins/pr/1707: Revert to poor man's logic Change all to comments to old comment style Fix comparing logic Remove spaces from tests check_swap: Fix perfdata und thresholds for big values and simplify code Introduce new perfdata functions and stuff for using (u)int64_t
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-MKD3-18/+56
2021-09-01Merge pull request #1704 from RincewindsHat/use_openbsd_netcat_for_debian_testsSven Nierlein1-1/+1
Use netcat-openbsd for debian explicitely
2021-09-01Use netcat-openbsd for debian explicitelyrefs/pull/1704/headrincewind1-1/+1
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-08-10Merge pull request #1700 from monitoring-plugins/feature_check_curlSven Nierlein1-8/+77
Feature check curl
2021-07-28Merge branch 'master' into feature_check_curlrefs/pull/1700/headAndreas Baumann3-5/+3
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-22Merge pull request #1699 from peternewman/patch-1Sven Nierlein1-1/+1
Re-attach a comment to where it actually belongs
2021-07-22Re-attach a comment to where it actually belongsrefs/pull/1699/headPeter Newman1-1/+1
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-05Add quoting for the remaining variablesrefs/pull/1692/headrincewind1-3/+3
2021-07-05Merge pull request #1691 from RincewindsHat/no-host-check-for-check_by_sshSven Nierlein1-2/+0
Let ssh decide if a host is valid, enables usage of ssh .config file
2021-07-04check_curl: changed to STATE_CRITICAL for infinite loops (-ffollow)Andreas Baumann1-2/+2
2021-07-02Add -a option to print all matching lines and -p and -e options for perl and ↵rincewind1-14/+37
extended RE
2021-07-02Add extended and perl regexrincewind1-2/+32
2021-07-02Apply shellcheckrincewind1-29/+29
2021-07-02Remove modified note, since this is a git repositoryrincewind1-2/+1
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-27Merge pull request #1689 from monitoring-plugins/feature_check_curlSven Nierlein1-0/+12
fixes check_curl: "CRITICAL - Cannot retrieve certificate subject."
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-24check_curl.c: bugfix: verify certificates option should not force SSL to be usedrefs/pull/1688/headBarak Shohat2-2/+2
2021-05-21Merge pull request #1686 from monitoring-plugins/feature_github_actionsSven Nierlein11-136/+194
Migrate to GitHub actions
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-20remove useless commentsSven Nierlein2-39/+3
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2021-05-19Prune container and volumesJacob Hansen1-1/+2
Rather than manully removing the tmpfs volume. Signed-off-by: Jacob Hansen <jhansen@op5.com>
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-19Disable IPv6 in testsJacob Hansen1-1/+1
Docker doesn't seem to have a working IPv6 stack by default. For now disable IPv6 tests. 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-05-18Use openssl generated certJacob Hansen1-2/+6
The SSL certificate generate using make-ssl-cert seems to not work well with a our plugins. Causing a "CN could not be found" error to be shown. Using an OpenSSL generated certifiticate, seems to work better. Signed-off-by: Jacob Hansen <jhansen@op5.com>