summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-12-07Merge pull request #1564 from jacobbaungard/check_cluster_arg_validationSven Nierlein1-0/+10
check_cluster.c: Added data argument validation.
2018-12-07check_icmp: Correctly set address_family on lookupJacob Hansen1-1/+2
If a hostname is supplied instead of an IP address, the automatic address family detection would fail to correctly set the IP protocol version (it would always be IPv6). We now supply AF_UNSPEC to getaddrinfo, which should then return the correct address family in the result. Signed-off-by: Jacob Hansen <jhansen@op5.com>
2018-12-07check_icmp: removed outcommented codeJacob Hansen1-7/+0
Signed-off-by: Jacob Hansen <jhansen@op5.com>
2018-12-07check_icmp: process protocol version args firstJacob Hansen1-10/+22
Detection of protocol version is in the previous patch implemented in the add_target() function, which is called when adding the -H command line argument. That means that if a protocal version argument (-4, -6) is added after the -H then the protocol version might be incorrectly set. This patch ensures that we first process the protocol version arguments, and then we process the rest of the arguments. Signed-off-by: Jacob Hansen <jhansen@op5.com>
2018-12-07check_icmp: Automatically detect IP protocolJacob Hansen1-26/+45
This patch automatically detects whether the protocol version is IPv4 or IPv6 All credits to: https://github.com/ghciv6 Signed-off-by: Jacob Hansen <jhansen@op5.com>
2018-12-07Fixed parameter handling after 01efbb2183d49c5082598d4799788fc385342f28Lars Michelsen1-38/+38
2018-12-07check_icmp: Add IPv6 supportLars Michelsen1-161/+330
This commit adds IPv6 capabilities to check_icmp. It is now possible to specify the address family using the arguments -4 (default) or -6. To make the change possible we had to move the argument parsing previous to creating the socket to be able to create it with the correct address family. This commit also fixes some gcc 4.9.2 compiler warnings. It has been tested with several current linux distributions (debian, ubuntu, rh, sles). This commit fixes monitoring-plugins/monitoring-plugins#1291
2018-12-07Merge pull request #1565 from sni/travis_run_plugin_root_testsSven Nierlein18-438/+212
Travis run plugin root tests
2018-12-05plugins: check_http: Increase regexp limitrefs/pull/1566/headTomas Mozes1-1/+1
2018-12-04tests: rework test parametersrefs/pull/1565/headSven Nierlein16-303/+107
there were 2 variants of calling getTestParameter: - parameter, description, default value - parameter, env value, default value, description, scope While scope was never actually used and having 2 names for the same value led to having 2 different entries in the cache file for the same configuration. This commit removes the variants and simplifies tests parameters by only using the first 3 parameter variant.
2018-12-04NPTest: unify whitespaceSven Nierlein1-73/+73
2018-12-04tests: skip some tests if Monitoring::Plugin::Range isn't availableSven Nierlein1-67/+36
The check_file_age uses Monitoring::Plugin::Range internally. Skip thoses tests if the module isn't available.
2018-12-04tests: enable plugins-root testsSven Nierlein1-0/+1
2018-11-29check_cluster.c: Added data argument validation.refs/pull/1564/headKostyantyn Hushchyn1-0/+10
Signed-off-by: Kostyantyn Hushchyn <kgushtin@op5.com>
2018-11-12set hostname in for CURLOPT_CURL to virtual hostname in case of SSL (for SNI ↵Andreas Baumann1-1/+1
to work)
2018-11-10Merge remote-tracking branch 'monitoring-plugins/pr/1560'Holger Weiss1-1/+11
* monitoring-plugins/pr/1560: Add new flag --show-body/-B to print body
2018-11-09Add new flag --show-body/-B to print bodyrefs/pull/1560/headTobias Wolf1-1/+11
This should help with figuring out ia problem at a glance when enabled for healthz endpoints on web APIs, for example. The content of the body can point to what the problem is and help with diagnostics. Fixes #1559
2018-11-06Merge remote-tracking branch 'monitoring-plugins/pr/1476'Holger Weiss5-5/+8023
* monitoring-plugins/pr/1476: test: Add proxy tests via proxy Allow user to specify HTTP method after proxy CONNECT
2018-11-06test: Add proxy tests via proxyrefs/pull/1476/headMarkus Frosch4-2/+8009
On Travis with a local squid
2018-11-06Allow user to specify HTTP method after proxy CONNECTMarkus Frosch1-3/+14
Simple format, avoids refactoring the CONNECT feature.
2018-10-29check_curl: code cleanupSven Nierlein1-11/+0
CURLOPT_RESOLVE is not required, since we do not verify certificates in any way.
2018-10-25check_curl: rewrite connect_to / host headersSven Nierlein2-28/+35
since CURLOPT_CONNECT_TO is only available in later curl versions, we do it the other way round now and set the url from the address we want to connect to and then set the host header accordingly.
2018-10-24check_curl: code cleanupSven Nierlein1-1/+2
2018-10-24check_curl: handle supplied port correctlySven Nierlein1-4/+6
if a port was given by -p, it should not be overruled by the port extracted from -H.
2018-10-24check_curl: fix builds if curl is not enabledSven Nierlein2-2/+2
2018-10-24travis: --enable-check-curl is no longer requiredSven Nierlein1-1/+1
2018-10-24cleanup gitignoreSven Nierlein1-6/+0
these files have been removed meanwhile
2018-10-24builds: alway put picohttpparser into the dist tarballSven Nierlein2-4/+1
substituting PICOHTTPPARSER_DIR only when curl gets build, leads to different dist tarballs depending on wether libcurl was available or not. This then breaks later builds from this tarball because of the missing files.
2018-10-24check_curl: do not build check_curl if curllibs are missingSven Nierlein1-4/+3
2018-10-23uriparser: do not reset flags if already setSven Nierlein1-3/+5
2018-10-23picohttpparser: add header to final tarballSven Nierlein1-1/+1
2018-10-23check_curl: do not use pkg-config if --with-uriparser was supplied by ↵Sven Nierlein1-7/+4
commandline if we run configure with --with-uriparser=... it should use that path or fail.
2018-10-23check_curl: enable by default if all requirements are metSven Nierlein1-25/+15
2018-10-22check_curl: unify performance dataSven Nierlein1-23/+61
and align them with check_http
2018-10-22check_curl: use same page length in performance data and text outputSven Nierlein1-5/+5
2018-10-22added some advanced tests for check_curlChristopher Odenbach1-19/+61
2018-10-22Modified virtual host and port behaviour to match check_http.Christopher Odenbach1-15/+28
2018-10-22check_curl: changed name of plugin in user-agent and added curl_version() to itAndreas Baumann1-3/+3
2018-10-22check_curl: removed some dead codeAndreas Baumann1-10/+0
2018-10-22check_curl: removed die in redirect with relative paths, simple cases (same ↵Andreas Baumann1-2/+0
protocol, same host) actually work
2018-10-22..I should really not test in test code :-)Andreas Baumann1-1/+0
2018-10-22check_curl: made HTTP message optionalAndreas Baumann2-3/+4
2018-10-22check-curl: have some fallbacks ready if pkg-config is missing for uriparser ↵Andreas Baumann1-0/+4
tests
2018-10-22Avoid working with free'ed memoryJean-François Rameau1-2/+2
2018-10-22fixed typo in check_curl error message (unparsable)Andreas Baumann1-1/+1
2018-10-22removed embedded uriparser library, added --with-uriparser configure optionAndreas Baumann30-8325/+175
to use uriparser library from the system
2018-10-22bumped coyright to 2018 in check_curl.cAndreas Baumann1-2/+2
2018-10-22updated bundled version of uriparser to 0.8.5Andreas Baumann21-34/+38
2018-10-22fixed initialization of server_url (freeing non-pointer in case of redirects)Andreas Baumann1-1/+3
2018-10-22check_curl: cleanupSven Nierlein1-18/+2
- remove obsolete todos - change exit code to unknown if regex exectute fails Signed-off-by: Sven Nierlein <sven@nierlein.de>