summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2016-11-17check_radius: Replace functions removed in radcliHolger Weiss4-22/+23
The radcli library no longer offers the rc_get_ipaddr(), rc_good_ipaddr(), and rc_own_ipaddress() functions.
2016-11-16check_radius: Fix rc_send_server() call for radcliHolger Weiss1-0/+4
With radcli, the rc_send_server() function expects an additional argument.
2016-11-15check_radius: Remove unused variableHolger Weiss1-1/+0
The radcli library doesn't define the ENV type, and we don't use it anyway.
2016-11-15This patch adds support for using the readcli lib.Jan Wagner3-17/+26
http://radcli.github.io/radcli/ (Closes #1437)
2016-11-13add openssl 1.1 supportrefs/pull/1443/headvagrant5-10/+19
changes: - CRYPTO_lock detection replaced in configure.ac. We don't use that function anywhere, so just replace it with the suggested one from https://wiki.openssl.org/index.php/Library_Initialization#Autoconf - OPENSSL_NO_SSL2 is no longer defined while ssl2 is not included. Set it ourself using the suggested openssl 1.1 version check from https://wiki.openssl.org/index.php/1.1_API_Changes#Backward_compatibility - openssl 1.1 sends a sigpipe if the connection is still open when calling SSL_shutdown(), so move the close before the shutdown. Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-11Merge pull request #1391 from lausser/patch-1refs/pull/1444/headSven Nierlein1-0/+4
If a web page contains a nul character, check_http reads the complete page but --string does not search beyond this character.
2016-11-11Revert "adopt http test to changed ssl expire date output"Sven Nierlein1-4/+4
This reverts commit 6cd50bc42cb4b25a3c0f7153df7f83b7262f404b.
2016-11-11tests: make tests reliable for forcing the locale to CSven Nierlein1-0/+2
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-11clean cpan installationSven Nierlein1-3/+0
HTTP::Daemon::SSL gets installed as pkg already.
2016-11-11update news fileSven Nierlein1-0/+1
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-11check_http: add host header testsSven Nierlein1-1/+27
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-11travis: install HTTP::Daemon::SSL perl moduleSven Nierlein1-0/+3
which is required for plugins/tests/check_http.t Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-11adopt http test to changed ssl expire date outputSven Nierlein1-4/+4
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-11patch to support the concept of virtual portsChristopher Odenbach2-8/+31
2016-11-08update thanks fileSven Nierlein1-0/+1
2016-11-08Merge pull request #1442 from waja/check_dig_use_retry_instead_triesSven Nierlein1-2/+2
check_dig: use +retry instead of +tries
2016-11-08Merge pull request #1441 from waja/check_ide_smart_remove_unused_functionSven Nierlein1-26/+0
check_ide_smart: remove unused function
2016-11-08check_dig: use +retry instead of +triesrefs/pull/1442/headChristian Kujau1-2/+2
After upgrading from an Ubuntu/15.10 to 16.04 installation, I noticed that check_dig is always returning a WARNING: $ /usr/lib/nagios/plugins/check_dig -l localhost -v /usr/bin/dig -p 53 @127.0.0.1 localhost A +tries=3 +time=6 Looking for: 'localhost' DNS WARNING - 0.008 seconds response time (dig returned an error status)|time=0.008274s;;;0.000000 The older Ubuntu installation got its check_dig from the nagios-plugins-standard package[0] which did not include the +tries option. The current Ubuntu version provides its check_dig from the monitoring-plugins-standard package[1], which _does_ use the +tries option that was introduced with df53473[2]. On my system, it so happens that /usr/bin/dig is provided not by the (BIND) dnsutils package but by knot-dnsutils[3] from the Knot DNS project. The Knot dig(1) command doesn't support the +tries option[4] but does support +retry (which is also supported[5] by the BIND dig(1) command). One way to fix that would be for me to install the BIND dnsutils package. But I did not want to do that: it's so much larger in size and pulls in much more dependencies than the knot-dnsutils package. The patch below changes check_dig to use +retry instead of +tries. Both options are similar, but not the same: +retry - Sets the number of times to retry UDP queries to server to T instead of the default, 2. Unlike +tries, this does not include the initial query As number_tries seems to be hard coded to 3, I've lowered DEFAULT_TRIES to 2 so check_dig should behave as before (with +tries=3). Thanks, Christian. [0] http://packages.ubuntu.com/wily/nagios-plugins-standard [1] http://packages.ubuntu.com/xenial/monitoring-plugins-standard [2] https://github.com/monitoring-plugins/monitoring-plugins/commit/df53473 [3] http://packages.ubuntu.com/xenial/knot-dnsutils [4] https://www.knot-dns.cz/docs/2.x/html/man_kdig.html#notes [5] https://ftp.isc.org/isc/bind9/cur/9.10/doc/arm/man.dig.html Signed-off-by: Christian Kujau <lists@nerdbynature.de>
2016-11-08check_ide_smart: remove unused functionrefs/pull/1441/headSebastian Herbszt1-26/+0
Remove function unused since commit b5cc292. Signed-off-by: Sebastian Herbszt <herbszt@gmx.de>
2016-11-07tests: enable snmp testsSven Nierlein1-2/+2
Enabled snmp tests against snmpd on localhost. It was installed already in the travis file, we just need to enable the tests by setting the parameters in the answers file. Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-07check_snmp: fix number of tests skipedSven Nierlein1-1/+1
2016-11-07check_snmp: add testcase for no datatypeSven Nierlein1-1/+5
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-07Revert "check_snmp: fix push request 1173 for SNMP responses without ↵Sven Nierlein1-1/+1
datatype indicator" This reverts commit 3178c8c0ff18822a04fe01c749f3564887473eed.
2016-11-07Revert "update thanks file"Sven Nierlein1-1/+0
This reverts commit 4403c402cba0315c2cb9ebe40639a5fe312d643e.
2016-11-07update thanks fileSven Nierlein1-0/+1
2016-11-07check_snmp: fix push request 1173 for SNMP responses without datatype indicatorThomas Kurschel1-1/+1
2016-11-07fix superfluous argument to printf (#fixes 1308)Sven Nierlein1-1/+1
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-07Merge pull request #1311 from waja/check_ntp_remove_unused_variablesSven Nierlein1-4/+2
check_ntp: remove unused variables
2016-11-07Merge pull request #1309 from waja/negate_remove_unused_variablesSven Nierlein1-2/+2
negate: remove unused variables
2016-11-07Merge pull request #1312 from waja/check_disk_remove_unused_variablesSven Nierlein1-7/+2
check_disk: remove unused variables
2016-11-07Merge pull request #1313 from waja/check_ntp_time_remove_unused_variablesSven Nierlein1-4/+2
check_ntp_time: remove unused variables
2016-11-07Merge pull request #1307 from waja/check_dns_dead_codeSven Nierlein1-6/+0
check_dns: remove dead code
2016-11-07update news and thanks fileSven Nierlein2-0/+3
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-07tests: adjust check_http to new output formatSven Nierlein1-2/+2
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-07GMT expiry displayPeter (pir) Radcliffe2-5/+14
Change solution to display GMT time in the local display format with the offset number of hours from GMT to be clear about what timezone this is if the local display format does not include offset.
2016-11-07Revert "Fix check_http test time output"Peter (pir) Radcliffe1-4/+4
This reverts commit 2d9e61a4382b8366331cde7617dbc4e381a0219a.
2016-11-07Fix check_http test time outputPeter (pir) Radcliffe1-4/+4
Tests need to match new output time and timezone.
2016-11-07Use GMT timezone in SSL certsPeter (pir) Radcliffe1-3/+3
SSL certs are required to use times in GMT per https://www.ietf.org/rfc/rfc5280.txt but the mktime() here assumes the current timezone. Fix the time_t conversion to be done assuming GMT with timegm() and only do it once rather than twice. Display the expiry date and time with ISO format years and give an offset from GMT and a timezone to be very clear about exactly what time is being displayed. Time given is correct and now in the machine’s timezone.
2016-11-07check_snmp.c - Added IPv6 supportabrist3-3/+15
The "-6" optarg now prepends the server_address with "udp6:" for the snmpget external command as per the net-snmp syntax at: http://www.net-snmp.org/wiki/index.php/FAQ:Applications_28 Thanks to DrydenK (Roberto Greiner) for the heads up.
2016-11-07check_users: add new test casesSven Nierlein1-1/+3
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-07check_users: still allow zero thresholdsSven Nierlein1-2/+2
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-11-07check_users not correctly detecting thresholdsJohn C. Frickson4-61/+75
Fix for issue https://github.com/nagios-plugins/nagios-plugins/issues/81 check_users now uses the standard warning and critical ranges parser and a standard perdata output routine.
2016-11-04check_disk - show all disks if state is ok and option error only is usedrefs/pull/1436/headOliver Skibbe1-4/+15
This fix changes output of check_disk in case of --error-only/-e option is used and state is ok - Old output: DISK OK - New output: DISK OK - free space: / 159731 MB (83% inode=61%); /dev/shm 2926 MB (100% inode=99%); /boot 58 MB (32% inode=99%); Resolves: #1420
2016-09-17check_by_ssh: print command output in verbose modeSven Nierlein1-0/+7
right now it is not possible to print the command output of ssh. check_by_ssh only prints the command itself. This patchs adds printing the output too. This makes it possible to use ssh with verbose logging which helps debuging any connection, key or other ssh problems. Note: you must use -E,--skip-stderr=<high number>, otherwise check_by_ssh would always exit with unknown state. Example: ./check_by_ssh -H localhost -o LogLevel=DEBUG3 -C "sleep 1" -E 999 -v Signed-off-by: Sven Nierlein <sven@nierlein.de>
2016-09-16check_dhcp: Make xid a bit more randomHolger Weiss1-3/+2
Don't just use the current time(2) to seed the PRNG that generates the xid. A DHCP server might be checked multiple times per second.
2016-09-14check_dhcp: Don't peek before reading responseHolger Weiss1-7/+1
Omit a recvfrom(3) call which has the MSG_PEEK flag set and should have no effect at all. According to a comment in the code, this call was meant to be a workaround to an opaque issue. If anyone runs into that issue, we should fix it properly instead.
2016-03-24Update AUTHORSOliver Skibbe1-0/+1
Added Oliver Skibbe to Authors
2016-02-22Merge branch 'pr/1386'Holger Weiss1-1/+1
* pr/1386: check_dig: expected answer is now incasesensitive
2016-02-22Let check_http test use HTTP/1.1Holger Weiss1-1/+1
2016-02-22Merge branch 'pr/1404'Holger Weiss1-1/+1
* pr/1404: DNS is case insensitive!