summaryrefslogtreecommitdiffstats
path: root/plugins
AgeCommit message (Collapse)AuthorFilesLines
2021-11-17check_snmp: fix performance thresholds when using multiple oidsSven Nierlein2-5/+21
when using check_snmp with multiple oids it simply printed the unparsed content from -w/-c into the thresholds for each oid. So each oid contained the hole -w from all oids. ./check_snmp ... -o iso.3.6.1.2.1.25.1.3.0,iso.3.6.1.2.1.25.1.5.0 -w '1,2' -c '3,4' before: SNMP ... | HOST-RESOURCES-MIB::hrSystemInitialLoadDevice.0=393216;1,2;3,4 HOST-RESOURCES-MIB::hrSystemNumUsers.0=24;1,2;3,4 after: SNMP ... | HOST-RESOURCES-MIB::hrSystemInitialLoadDevice.0=393216;1;3 HOST-RESOURCES-MIB::hrSystemNumUsers.0=24;2;4 This also applies to fixed thresholds since check_snmp translates negative infinities from: '~:-1' to '@-1:~'
2021-11-11Merge pull request #1715 from ghen2/lmtpRincewindsHat1-4/+14
check_smtp: add -L flag to support LMTP (LHLO instead of HELO/EHLO).
2021-11-01check_procs: add test for elapsed timeSven Nierlein2-2/+230
2021-11-01remove unused test fileSven Nierlein1-84/+0
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2021-11-01check_procs: improve ps args autodetectionSven Nierlein1-4/+7
one of the first ps commands in the configure.ac is `axwo 'stat comm vsz rss user uid pid ppid args'` which works on most modern linux systems (checked debian 10/11 and centos 7/8). But this test misses the etime argument. Therefore `check_procs --metric=ELAPSED` does not work. To fix this, we simply do the same test including etime before that one. Signed-off-by: Sven Nierlein <sven@nierlein.de>
2021-11-01check_procs: remove trailing whitespaceSven Nierlein1-25/+25
2021-10-31check_mysql.c: Detect running mysqldumpGerardo Malazdrewicz1-6/+26
When checking a slave, if the IO Thread or the SQL Thread are stopped, check for running mysqldump threads, return STATE_OK if there is any. Requires PROCESS privilege to work (else the mysqldump thread(s) would not be detected). Enlarged SLAVERESULTSIZE to fit "Mysqldump: in progress" at the end of the string. Got a NULL pointer in row[seconds_behind_field] instead of the "NULL" string when a mysqldump is running [mysql 5.7.34 + libmariadb3 10.3.31], so added a check for that.
2021-10-28check_smtp: add -L flag to support LMTP (LHLO instead of HELO/EHLO).refs/pull/1715/headGeert Hendrickx1-4/+14
2021-10-27Replace tabs with spaces and do some formattingrefs/pull/1714/headRincewindsHat1-55/+54
2021-10-27Remove whitespace at the end of linesRincewindsHat1-17/+17
2021-10-26Fix my own errors for real this time and add some comments, so I wont do ↵RincewindsHat1-32/+32
them again
2021-10-26Re-add separation semicolons for perfdata, which I deletedRincewindsHat1-8/+16
2021-10-26Reform some arithmetical operations for more clarityRincewindsHat1-7/+9
2021-10-26hopefully fix warning high tide with percentageRincewindsHat1-1/+1
2021-10-26Fix some testsRincewindsHat1-3/+4
2021-10-26Remove useless lines and fix some formattingRincewindsHat1-12/+8
2021-10-26Switch to uint64_tRincewindsHat1-13/+11
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-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-24check_curl.c: bugfix: verify certificates option should not force SSL to be usedrefs/pull/1688/headBarak Shohat2-2/+2
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.