From noreply at sourceforge.net Sun Jul 1 20:55:12 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 11:55:12 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539317 ] potential typo in the --help text of check_apt Message-ID: Bugs item #3539317, was opened at 2012-07-01 11:55 Message generated for change (Tracker Item Submitted) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539317&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Interface (example) Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) Summary: potential typo in the --help text of check_apt Initial Comment: From: http://tracker.nagios.org/view.php?id=299 Hi. The --help text of the check_apt check says: > -i, --include=REGEXP > Include only packages matching REGEXP. Can be specified multiple times > the values will be combined together. Any patches matching this list > cause the plugin to return WARNING status. Others will be ignored. > Default is to include all packages. I guess in the second line, this should read "Any packages" instead of "Any patches". Cheers, Chris ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539317&group_id=29880 From noreply at sourceforge.net Sun Jul 1 21:00:51 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 12:00:51 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539319 ] check_apt should check whether ALL repositories are availabl Message-ID: Bugs item #3539319, was opened at 2012-07-01 12:00 Message generated for change (Tracker Item Submitted) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539319&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) Summary: check_apt should check whether ALL repositories are availabl Initial Comment: From: http://tracker.nagios.org/view.php?id=300 Hi. Marking this as major as it's security relevant. Regardless of whether one uses check_apt for doing real upgrading, or only for checking (--no-upgrade), it's crucial that all configured (via sources.list and friends) repositories are "loaded" and "recent" enough. Otherwise an attacker could just block updates (of the package lists) or do downgrade attacks. As far as I can see the following must be checked: a) For each repository configured in sources.list AND any other sources location (e.g. sources.list.d/*) the necessary Release files must be present in /var/lib/apt/lists/ . I hope the presence of a Release file or a specific repository means, that all related files (Packages files, etc.) have been downloaded and verified (-> signatures). (Are there race condition issues, when an automatic package list upgrade happens while the check_apt takes place?) If any is missing, this means that repos are missing and a blocking attack could be happening. b) For ALL Release files, the current date must be between, these values within the file: Date: Sun, 18 Mar 2012 12:12:06 UTC Valid-Until: Wed, 28 Mar 2012 12:12:06 UTC If not, a downgrade attack could be in place. If a) or b) is true, the check's result should be CRITICAL, with a warning on what has happened. Perhaps a options should be added to disable this checks, but I think it would be dangerous. Better not provide them or at least warn aloud from their usage. Cheers, Chris. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539319&group_id=29880 From noreply at sourceforge.net Sun Jul 1 21:02:19 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 12:02:19 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539319 ] check_apt should check whether ALL repositories are availabl Message-ID: Bugs item #3539319, was opened at 2012-07-01 12:00 Message generated for change (Comment added) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539319&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. >Category: General plugin execution Group: None Status: Open Resolution: None >Priority: 9 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) Summary: check_apt should check whether ALL repositories are availabl Initial Comment: From: http://tracker.nagios.org/view.php?id=300 Hi. Marking this as major as it's security relevant. Regardless of whether one uses check_apt for doing real upgrading, or only for checking (--no-upgrade), it's crucial that all configured (via sources.list and friends) repositories are "loaded" and "recent" enough. Otherwise an attacker could just block updates (of the package lists) or do downgrade attacks. As far as I can see the following must be checked: a) For each repository configured in sources.list AND any other sources location (e.g. sources.list.d/*) the necessary Release files must be present in /var/lib/apt/lists/ . I hope the presence of a Release file or a specific repository means, that all related files (Packages files, etc.) have been downloaded and verified (-> signatures). (Are there race condition issues, when an automatic package list upgrade happens while the check_apt takes place?) If any is missing, this means that repos are missing and a blocking attack could be happening. b) For ALL Release files, the current date must be between, these values within the file: Date: Sun, 18 Mar 2012 12:12:06 UTC Valid-Until: Wed, 28 Mar 2012 12:12:06 UTC If not, a downgrade attack could be in place. If a) or b) is true, the check's result should be CRITICAL, with a warning on what has happened. Perhaps a options should be added to disable this checks, but I think it would be dangerous. Better not provide them or at least warn aloud from their usage. Cheers, Chris. ---------------------------------------------------------------------- >Comment By: C?lestyo (calestyo) Date: 2012-07-01 12:02 Message: One addition: Another option should be added, that allow the user to narrow down (but NOT to enlarge!!) the interval given by Date/Valid-Until, when checking whether the current time is in between. The current interval seems to be quite large. Typically (but at least for testing/unstable) the Release files/etc are generated far more often than just once in the interval. So you'll get new files say daily and therefore it should be possible to narrow the interval down on the client side. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539319&group_id=29880 From noreply at sourceforge.net Sun Jul 1 21:10:55 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 12:10:55 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539323 ] please include check_yum Message-ID: Bugs item #3539323, was opened at 2012-07-01 12:10 Message generated for change (Tracker Item Submitted) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539323&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) Summary: please include check_yum Initial Comment: (Roughly) From: http://tracker.nagios.org/view.php?id=306 Hi. It's good that check_apt is already shipped as part of the nagios-plugins. YUM is the current higher level package manager for all RedHat based distros, i.e. the majors RHEL (and its clones, like CentOS and Scientific Linux) and Fedora. Therefore (because of its importance and wide spread), may I propose to include check_yum, too. The official site for check_yum is: https://code.google.com/p/check-yum/ The Nagios Exchange site is: http://exchange.nagios.org/directory/Plugins/Operating-Systems/Linux/Check_Yum/details Cheers. Chris. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539323&group_id=29880 From noreply at sourceforge.net Sun Jul 1 21:27:21 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 12:27:21 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539328 ] check_apt: clarify that it doesn't upgrade per default Message-ID: Bugs item #3539328, was opened at 2012-07-01 12:27 Message generated for change (Tracker Item Submitted) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539328&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Interface (example) Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) Summary: check_apt: clarify that it doesn't upgrade per default Initial Comment: (Roughly) From: http://tracker.nagios.org/view.php?id=302 Hi. check_apt uses its "--upgrade" option per default: -U, --upgrade=OPTS [Default] Perform an upgrade. If an optional OPTS argument is provided, apt-get will be run with these command line options instead of the default (-o 'Debug::NoLocking=true' -s -qq). Note that you may be required to have root privileges if you do not use the default options. I was quite confused by this, given the meanings of upgrade (install new packages) and update (get information about new packages) in Debian. The following seems to be true: - Using the upgrade mode in APT (and therefore the --upgrade option to the plugin) is really required, even to just get a list of upgradable packages. So this really needs to be the default. - No upgrades (in the sense of installing packags) are however executed, given the options (-o 'Debug::NoLocking=true' -s -qq). This is true for both, normal users (obviously) and root. In order to clear things up, I'd suggest to add a notes that: - the --upgrade option of the plugins (with it's default OPTS) doesn't do package upgrades (in the sense of installing new ones), neither as root nor as normal user. and optionally: - the --update option (and upgrade mode of APT) is needed even to just get a list of upgradable pacakges. Cheers, Chris. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539328&group_id=29880 From noreply at sourceforge.net Sun Jul 1 21:43:19 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 12:43:19 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539330 ] improve default APT options for check_apt Message-ID: Bugs item #3539330, was opened at 2012-07-01 12:43 Message generated for change (Tracker Item Submitted) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539330&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: General plugin execution Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) Summary: improve default APT options for check_apt Initial Comment: (Roughly) From: http://tracker.nagios.org/view.php?id=303 Hi. check_apt uses the following default options on update: -q Unlike the default options for --upgrade these are however undocumented in --help, which I suggest to add. Cheers, Chris btw: I'd write patches. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539330&group_id=29880 From noreply at sourceforge.net Sun Jul 1 21:44:05 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 12:44:05 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539330 ] check_apt: document default OPTs for --update Message-ID: Bugs item #3539330, was opened at 2012-07-01 12:43 Message generated for change (Settings changed) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539330&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. >Category: Interface (example) Group: None Status: Open Resolution: None >Priority: 3 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) >Summary: check_apt: document default OPTs for --update Initial Comment: (Roughly) From: http://tracker.nagios.org/view.php?id=303 Hi. check_apt uses the following default options on update: -q Unlike the default options for --upgrade these are however undocumented in --help, which I suggest to add. Cheers, Chris btw: I'd write patches. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539330&group_id=29880 From noreply at sourceforge.net Sun Jul 1 21:46:09 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Sun, 01 Jul 2012 12:46:09 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3539328 ] check_apt: clarify that it doesn't upgrade per default Message-ID: Bugs item #3539328, was opened at 2012-07-01 12:27 Message generated for change (Comment added) made by calestyo You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539328&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Interface (example) Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: C?lestyo (calestyo) Assigned to: Nobody/Anonymous (nobody) Summary: check_apt: clarify that it doesn't upgrade per default Initial Comment: (Roughly) From: http://tracker.nagios.org/view.php?id=302 Hi. check_apt uses its "--upgrade" option per default: -U, --upgrade=OPTS [Default] Perform an upgrade. If an optional OPTS argument is provided, apt-get will be run with these command line options instead of the default (-o 'Debug::NoLocking=true' -s -qq). Note that you may be required to have root privileges if you do not use the default options. I was quite confused by this, given the meanings of upgrade (install new packages) and update (get information about new packages) in Debian. The following seems to be true: - Using the upgrade mode in APT (and therefore the --upgrade option to the plugin) is really required, even to just get a list of upgradable packages. So this really needs to be the default. - No upgrades (in the sense of installing packags) are however executed, given the options (-o 'Debug::NoLocking=true' -s -qq). This is true for both, normal users (obviously) and root. In order to clear things up, I'd suggest to add a notes that: - the --upgrade option of the plugins (with it's default OPTS) doesn't do package upgrades (in the sense of installing new ones), neither as root nor as normal user. and optionally: - the --update option (and upgrade mode of APT) is needed even to just get a list of upgradable pacakges. Cheers, Chris. ---------------------------------------------------------------------- >Comment By: C?lestyo (calestyo) Date: 2012-07-01 12:46 Message: I would write the patches, if the ideas themselves are accepted. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3539328&group_id=29880 From nagios at flatto.net Mon Jul 2 10:05:02 2012 From: nagios at flatto.net (Assaf Flatto) Date: Mon, 02 Jul 2012 09:05:02 +0100 Subject: [Nagiosplug-devel] check_http support of client SSL certificates Message-ID: <4FF1562E.8020200@flatto.net> Hello I am working on project where the internal web servers require a client SSL certificate to communicate and query any of the pages. We are using a bespoke curl script to import and perform the queries, but I wanted to know if there is a roadmap for adding support for client certificates to the check_http plugin ? Regards Assaf Flatto From misterrody at gmail.com Wed Jul 4 11:54:28 2012 From: misterrody at gmail.com (Kandri Rody Yahya) Date: Wed, 4 Jul 2012 11:54:28 +0200 Subject: [Nagiosplug-devel] check_snmp_mem.pl Message-ID: Merci de me dire une solution pour: check_snmp_mem.pl v$Revision: 1300 $ (nagios-plugins 1.4.15) The nagios plugins come with ABSOLUTELY NO WARRANTY. You may redistribute copies of the plugins under the terms of the GNU General Public License. For more information about these matters, see the file named COPYING. Scan arbitrary log files for regular expression matches. Usage: check_snmp_mem.pl -l -s -p [-n ] -c -- - - - - - - - - - - - - - - - - - - Yahya Kandri Rody: 00 212 6 67 74 71 31 00 33 6 52 42 75 81 - - - - - - - - - - - - - - - - - - - -------------- next part -------------- An HTML attachment was scrubbed... URL: From holger at CIS.FU-Berlin.DE Thu Jul 5 10:32:48 2012 From: holger at CIS.FU-Berlin.DE (Holger =?iso-8859-1?Q?Wei=DF?=) Date: Thu, 5 Jul 2012 10:32:48 +0200 Subject: [Nagiosplug-devel] New plugin: check_dbi Message-ID: <20120705083248.GR18382890@CIS.FU-Berlin.DE> Yesterday, we've imported the check_dbi plugin written by Sebastian Harl, which allows for checking databases using DBI. It's available in the current snapshot: http://nagiosplug.sourceforge.net/snapshot/nagios-plugins-master.tar.gz Most probably, the plugin will be included with the next release. Until then, we can modify its behaviour without having to worry about backwards compatibility. So, it would be nice if anyone interested in this plugin could give the snapshot a try. Please send any feedback to the Plugins Development list: https://lists.sourceforge.net/lists/listinfo/nagiosplug-devel Many thanks to Sebastian for contributing this nice plugin! Holger From sh at teamix.net Thu Jul 5 11:20:41 2012 From: sh at teamix.net (Sebastian Harl) Date: Thu, 5 Jul 2012 11:20:41 +0200 Subject: [Nagiosplug-devel] [PATCH/PULL] New plugin: check_dbi In-Reply-To: References: <20110408094511.GJ20370@chough.tokkee.org> <20120606095512.GP10633@chough.tokkee.org> <20120606103842.GA16045381@CIS.FU-Berlin.DE> <20120608093033.GR10633@chough.tokkee.org> Message-ID: <20120705092041.GE9887@whisky.of.teamix.net> Hi hassan, On Mon, Jun 11, 2012 at 01:27:17PM +0500, hassan nashrah wrote: > I have a test case, that you may like to test. as well as if your plugin could do this it wud be grt. i have been looking for a while to do the following: > > My Oracle DB server details > > operating System: Solaris > > database: oracle > IP: 192.168.1.2 (not real ip) > > My Nagios Server details > OS : Ubuntu > i have configured nagios 3.x > > i connect to my db server from my nagios as follow: > > ssh -Y root at 192.168.1.2 > Password: xxxxxxxxx > [root at vboxdb /]# su - oracle > > Sun Microsystems Inc. SunOS 5.10 Generic January 2005 > [oracle at vboxdb ~]$ sqlplus / as sysdba > > SQL*Plus: Release 11.2.0.1.0 Production on Wed May 30 04:53:48 2012 > > Copyright (c) 1982, 2009, Oracle. All rights reserved. Well, this is not how check_dbi would connect to an Oracle database. Rather, you will need an 'oracle' driver for libdbi. According to the website somebody is working on such a thing but it seems to be incomplete and I don't know anything about the current status or whether it's usable at all, I'm afraid. > what i want to do i will list below ; > > SQL> select current_utilization, max_utilization, limit_value from v$resource_limit where resource_name =?sessions?; > > CURRENT_UTILIZATION MAX_UTILIZATION LIMIT_VALUE > > ??????- ????? ????????- ????? > > 386 404 1528 > > if the current utilization reaches 1500 alert critical, 1450 warning state. > else normal. > > can you tell me how i can use your plugin to do this. This is exactly what check_dbi is meant for. Please note though, that you can (currently) only check a single value only. The plugin uses the first (CURRENT_UTILIZATION) and simply ignores all other values. > by the way i've tried to compile your plugin. but i am getting errors > saying common.h not found. Could you please provide more information about that error, like the full compiler output? common.h is shipped with nagios-plugins, so I suppose there is some other error. HTH, Sebastian -- Sebastian Harl Consultant, Trainer teamix GmbH S?dwestpark 35 90449 N?rnberg fon: +49 911 30999-0 fax: +49 911 30999-99 mail: sh at teamix.de web: http://www.teamix.de Amtsgericht N?rnberg, HRB 18320 Gesch?ftsf?hrer: Oliver K?gow, Richard M?ller -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: From sh at teamix.net Thu Jul 5 11:53:55 2012 From: sh at teamix.net (Sebastian Harl) Date: Thu, 5 Jul 2012 11:53:55 +0200 Subject: [Nagiosplug-devel] [PATCH/PULL] Various improvements and fixes for check_pgsql In-Reply-To: <20110408093132.GH20370@chough.tokkee.org> References: <20110408093132.GH20370@chough.tokkee.org> Message-ID: <20120705095354.GF9887@whisky.of.teamix.net> Hi, while going through my nagios-plugins repo, I noticed the following branch still sitting there ;-) While the query support is a bit redundant with the check_dbi plugin by now, I still think that could also be included in check_pgsql (after all the code is already there, it's not that complex/much and I'm willing to support it ;-)). Comments, flames, whatever about that would be much appreciated, though. If people disagree, I'll extract the other patches. The branch is now available at . Any comments, feedback? Cheers, Sebastian On Fri, Apr 08, 2011 at 11:31:32AM +0200, Sebastian Harl wrote: > I'm not sure about the preferred way to submit patches, so I'll just go > for a "pull request" for now. Please feel free to beat me to whatever > you prefer most ;-) > > Anyway ? I've got a few changes to the check_pgsql plugin sitting in the > sh/check_pgsql branch at git://oss.teamix.org/nagiosplug.git: > > check_pgsql: Added support for executing queries. > check_pgsql: Fixed query perfdata output for empty warn/crit ranges. > check_pgsql: Use PQconnectdb() rather than PQsetdbLogin(). > check_pgsql: Allow UNIX socket directories as hostname as well. > check_pgsql: Removed -4/-6 flags from help output. > check_pgsql: Added support for the -o command line option. > check_pgsql: Updated copyright. > check_pgsql: Leave 'min' value in query perfdata empty. > check_pgsql: Determine connection time in ?s-resolution. > > The most important changes are the following: > > check_pgsql: Added support for executing queries. > > The query result (the double value of the first column in the first row, to be > precise) will be checked against threshold ranges specified using the -C and > -W options. > > Note that this also allows to query PostgreSQL internal values using the > information available from the database daemon's "statistics collector" -- see > the chapter "Monitoring Database Activity" in the PostgreSQL manual for > details. > > (diff: ) > > check_pgsql: Added support for the -o command line option. > > This option may be used to specify further connection parameters to be passed > to PQconnectdb(). For example, this may be used to specify a service name in > pg_service.conf to be used for additional connection parameters: -o > 'service=' or to specify the SSL mode: -o 'sslmode=require'. > > See the chapter "libpq - C Library" in the PostgreSQL manual for details. > > (diff: ) > > I'd be happy for any feedback and comments. > > Cheers, > Sebastian > -- Sebastian Harl Consultant, Trainer teamix GmbH S?dwestpark 35 90449 N?rnberg fon: +49 911 30999-0 fax: +49 911 30999-99 mail: sh at teamix.de web: http://www.teamix.de Amtsgericht N?rnberg, HRB 18320 Gesch?ftsf?hrer: Oliver K?gow, Richard M?ller -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: From sh at teamix.net Thu Jul 5 11:55:39 2012 From: sh at teamix.net (Sebastian Harl) Date: Thu, 5 Jul 2012 11:55:39 +0200 Subject: [Nagiosplug-devel] [PATCH/PULL] check_procs: output performance data In-Reply-To: <20110408093651.GI20370@chough.tokkee.org> References: <20110408093651.GI20370@chough.tokkee.org> Message-ID: <20120705095539.GG9887@whisky.of.teamix.net> Hi again, and another rediscovered branch. This is now available at . Any comments about the backward-incompatible changes? Cheers, Sebastian On Fri, Apr 08, 2011 at 11:36:51AM +0200, Sebastian Harl wrote: > Hi again, > > I've got two patches sitting in the sh/check_procs branch at > git://oss.teamix.org/nagiosplug.git that add support for perfdata to > check_procs: > > check_procs: Use the range/threshold support functions from libnagiosplug. > check_procs: Added performance data to plugin output. > > The changes are rather small, so here's a diff for both patches: > > > Note: Previously, the compatibility code for 'check_procs ' > accepted something like 'check_procs -w 10:-1 -c 10:-1 20 50' as well > (treating it as if '-w 10:20 -c 10:50' was specified). This is no longer > the case ? additional arguments are only used as warn/crit thresholds in > case -w/-c is not specified at all. > > Looking forward to comments and suggestions ;-) > > Cheers, > Sebastian > -- Sebastian Harl Consultant, Trainer teamix GmbH S?dwestpark 35 90449 N?rnberg fon: +49 911 30999-0 fax: +49 911 30999-99 mail: sh at teamix.de web: http://www.teamix.de Amtsgericht N?rnberg, HRB 18320 Gesch?ftsf?hrer: Oliver K?gow, Richard M?ller -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: From william at leibzon.org Thu Jul 5 13:38:26 2012 From: william at leibzon.org (William Leibzon) Date: Thu, 5 Jul 2012 04:38:26 -0700 Subject: [Nagiosplug-devel] New plugin: check_dbi In-Reply-To: <20120705083248.GR18382890@CIS.FU-Berlin.DE> References: <20120705083248.GR18382890@CIS.FU-Berlin.DE> Message-ID: Because it requires additional libraries which not everyone may have, please make its inclusion optional as a configure option (and don't forget to actually check for libraries with configure as well). On Thu, Jul 5, 2012 at 1:32 AM, Holger Wei? wrote: > Yesterday, we've imported the check_dbi plugin written by Sebastian > Harl, which allows for checking databases using DBI. It's available in > the current snapshot: > > http://nagiosplug.sourceforge.net/snapshot/nagios-plugins-master.tar.gz > > Most probably, the plugin will be included with the next release. Until > then, we can modify its behaviour without having to worry about > backwards compatibility. So, it would be nice if anyone interested in > this plugin could give the snapshot a try. Please send any feedback to > the Plugins Development list: > > https://lists.sourceforge.net/lists/listinfo/nagiosplug-devel > > Many thanks to Sebastian for contributing this nice plugin! > > Holger > > > ------------------------------------------------------------------------------ > Live Security Virtual Conference > Exclusive live event will cover all the ways today's security and > threat landscape has changed and how IT managers can respond. Discussions > will include endpoint security, mobile security and the latest in malware > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ > _______________________________________________________ > Nagios Plugin Development Mailing List > Nagiosplug-devel at lists.sourceforge.net > Unsubscribe at > https://lists.sourceforge.net/lists/listinfo/nagiosplug-devel > ::: Please include plugins version (-v) and OS when reporting any issue. > ::: Messages without supporting info will risk being sent to /dev/null > -------------- next part -------------- An HTML attachment was scrubbed... URL: From holger at CIS.FU-Berlin.DE Thu Jul 5 14:03:17 2012 From: holger at CIS.FU-Berlin.DE (Holger =?iso-8859-1?Q?Wei=DF?=) Date: Thu, 5 Jul 2012 14:03:17 +0200 Subject: [Nagiosplug-devel] New plugin: check_dbi In-Reply-To: References: <20120705083248.GR18382890@CIS.FU-Berlin.DE> Message-ID: <20120705120316.GT18382890@CIS.FU-Berlin.DE> * William Leibzon [2012-07-05 04:38]: > Because it requires additional libraries which not everyone may have, > please make its inclusion optional as a configure option (and don't forget > to actually check for libraries with configure as well). Just as any of our plugins which require libraries, the plugin will be built if the configure script finds them, otherwise it won't be built. Or did you get an error message while building the snapshot? Holger From william at leibzon.org Thu Jul 5 14:12:29 2012 From: william at leibzon.org (William Leibzon) Date: Thu, 5 Jul 2012 05:12:29 -0700 Subject: [Nagiosplug-devel] New plugin: check_dbi In-Reply-To: <20120705120316.GT18382890@CIS.FU-Berlin.DE> References: <20120705083248.GR18382890@CIS.FU-Berlin.DE> <20120705120316.GT18382890@CIS.FU-Berlin.DE> Message-ID: On Thu, Jul 5, 2012 at 5:03 AM, Holger Wei? wrote: > * William Leibzon [2012-07-05 04:38]: > > Because it requires additional libraries which not everyone may have, > > please make its inclusion optional as a configure option (and don't > forget > > to actually check for libraries with configure as well). > > Just as any of our plugins which require libraries, the plugin will be > built if the configure script finds them, otherwise it won't be built. > Or did you get an error message while building the snapshot? Haven't checked it yet actually. And I don't have the dbi library on my dev system actually and have been struggling with inclusion of newer ssl library to properly build. I need to upgrade soon. Wiliam -------------- next part -------------- An HTML attachment was scrubbed... URL: From noreply at sourceforge.net Fri Jul 6 13:23:12 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2012 04:23:12 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Patches-2430999 ] add check certificate support for LDAP plugin Message-ID: Patches item #2430999, was opened at 2008-12-15 05:45 Message generated for change (Comment added) made by guillomovitch You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397599&aid=2430999&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Guillaume Rousse (guillomovitch) Assigned to: Nobody/Anonymous (nobody) Summary: add check certificate support for LDAP plugin Initial Comment: Here's a patch adding the certificate expiration check feature to the ldap plugin, based on the http plugin. For some unknown reason, probably due to the way secure connection is established, it doesn't work with the --ssl options, only for the --starttls one. ---------------------------------------------------------------------- >Comment By: Guillaume Rousse (guillomovitch) Date: 2012-07-06 04:23 Message: Here is yet another update of this patch. BTW, this is a pending issue for 4 years now, I'd appreciate a minimal feedback from developpers... ---------------------------------------------------------------------- Comment By: Guillaume Rousse (guillomovitch) Date: 2008-12-21 12:10 Message: File Added: nagios-plugins-1.4.13-check_ldap_certificate.patch ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397599&aid=2430999&group_id=29880 From noreply at sourceforge.net Fri Jul 6 13:53:07 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2012 04:53:07 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Patches-2430999 ] add check certificate support for LDAP plugin Message-ID: Patches item #2430999, was opened at 2008-12-15 05:45 Message generated for change (Settings changed) made by hweiss You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397599&aid=2430999&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Guillaume Rousse (guillomovitch) >Assigned to: Holger Weiss (hweiss) Summary: add check certificate support for LDAP plugin Initial Comment: Here's a patch adding the certificate expiration check feature to the ldap plugin, based on the http plugin. For some unknown reason, probably due to the way secure connection is established, it doesn't work with the --ssl options, only for the --starttls one. ---------------------------------------------------------------------- Comment By: Guillaume Rousse (guillomovitch) Date: 2012-07-06 04:23 Message: Here is yet another update of this patch. BTW, this is a pending issue for 4 years now, I'd appreciate a minimal feedback from developpers... ---------------------------------------------------------------------- Comment By: Guillaume Rousse (guillomovitch) Date: 2008-12-21 12:10 Message: File Added: nagios-plugins-1.4.13-check_ldap_certificate.patch ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397599&aid=2430999&group_id=29880 From noreply at sourceforge.net Fri Jul 6 13:54:18 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Fri, 06 Jul 2012 04:54:18 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Patches-2430999 ] add check certificate support for LDAP plugin Message-ID: Patches item #2430999, was opened at 2008-12-15 05:45 Message generated for change (Comment added) made by hweiss You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397599&aid=2430999&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Guillaume Rousse (guillomovitch) Assigned to: Holger Weiss (hweiss) Summary: add check certificate support for LDAP plugin Initial Comment: Here's a patch adding the certificate expiration check feature to the ldap plugin, based on the http plugin. For some unknown reason, probably due to the way secure connection is established, it doesn't work with the --ssl options, only for the --starttls one. ---------------------------------------------------------------------- >Comment By: Holger Weiss (hweiss) Date: 2012-07-06 04:54 Message: Thank you for your patch, and sorry for the long delay. I'll look into this before the next release. ---------------------------------------------------------------------- Comment By: Guillaume Rousse (guillomovitch) Date: 2012-07-06 04:23 Message: Here is yet another update of this patch. BTW, this is a pending issue for 4 years now, I'd appreciate a minimal feedback from developpers... ---------------------------------------------------------------------- Comment By: Guillaume Rousse (guillomovitch) Date: 2008-12-21 12:10 Message: File Added: nagios-plugins-1.4.13-check_ldap_certificate.patch ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397599&aid=2430999&group_id=29880 From saikrishna.pogula at gmail.com Mon Jul 9 12:14:41 2012 From: saikrishna.pogula at gmail.com (sai krishna) Date: Mon, 9 Jul 2012 15:44:41 +0530 Subject: [Nagiosplug-devel] Nagios error In-Reply-To: References: Message-ID: Hi , > I am sending screen shots what i am getting,could you pls help me > out.. > -------------- next part -------------- An HTML attachment was scrubbed... URL: From john.knepper at dandsltd.com Wed Jul 18 02:57:45 2012 From: john.knepper at dandsltd.com (John Knepper) Date: Tue, 17 Jul 2012 19:57:45 -0500 Subject: [Nagiosplug-devel] Help with C plugin: Message-ID: Hi, I am currently trying to develop a plugin in C, and it runs fine on the command-line, but Nagios is returning (null) as its output. After research, and also reviewing the following article on developing plugins with C ( http://nagiosplug.sourceforge.net/developer-guidelines.html#SYSCMDAUXFILES) I found that it is because I lack the spopen() function; I think? I have tried to include this in my C source, but I keep getting errors when trying to compile it such as conflicts with some functions from the standard library headers like string.h. Is there a compiler argument I need to use in particular when compiling my plugin? Thanks! Best, John -------------- next part -------------- An HTML attachment was scrubbed... URL: From mlacorte365 at gmail.com Wed Jul 18 22:49:16 2012 From: mlacorte365 at gmail.com (Michael LaCorte) Date: Wed, 18 Jul 2012 13:49:16 -0700 Subject: [Nagiosplug-devel] 1.4.16: Error in check_http.c when compiling without SSL Message-ID: Hello, I was compiling the latest release of Nagios Plugins when I ran into a compilation issue. Make was returning the following error when trying to compile check_http.c (v1.4.16): check_http.c:312:9: error: ?ssl_version? undeclared (first use in this function) After looking at the file, I noticed that 'ssl_version' variable was declared within an '#ifdef HAVE_SSL'. After moving the 'ssl_version' declaration outside of the '#ifdef', it compiled fine. I couldn't find anything about this on Google, so I'm sending you this email to make sure you are aware of the issue. Thanks, Michael LaCorte -------------- next part -------------- An HTML attachment was scrubbed... URL: From holger at CIS.FU-Berlin.DE Wed Jul 18 23:48:10 2012 From: holger at CIS.FU-Berlin.DE (Holger =?iso-8859-1?Q?Wei=DF?=) Date: Wed, 18 Jul 2012 23:48:10 +0200 Subject: [Nagiosplug-devel] 1.4.16: Error in check_http.c when compiling without SSL In-Reply-To: References: Message-ID: <20120718214810.GD24006341@CIS.FU-Berlin.DE> * Michael LaCorte [2012-07-18 13:49]: > I was compiling the latest release of Nagios Plugins when I ran into a > compilation issue. Make was returning the following error when trying > to compile check_http.c (v1.4.16): > > check_http.c:312:9: error: 'ssl_version' undeclared (first use in this function) > > After looking at the file, I noticed that 'ssl_version' variable was > declared within an '#ifdef HAVE_SSL'. Thanks for the note, I've now fixed this in the current Git code. Holger From noreply at sourceforge.net Thu Jul 19 11:29:08 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Thu, 19 Jul 2012 02:29:08 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3545799 ] check_snmp always returns OK Message-ID: Bugs item #3545799, was opened at 2012-07-19 02:29 Message generated for change (Tracker Item Submitted) made by tj8 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3545799&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Argument proccessing Group: release-1.4.15 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Tim (tj8) Assigned to: Nobody/Anonymous (nobody) Summary: check_snmp always returns OK Initial Comment: just upgrade plugins to nagios-plugins-1.4.15_1 when i try and use the command: ./check_snmp -H testserver -o enterprises.3032.64.4.1.2.4.101.120.105.109.1 -C public -w 100 -v I get: /usr/local/bin/snmpget -t 1 -r 5 -m ALL -v 1 [authpriv] testserver:161 enterprises.3032.64.4.1.2.4.101.120.105.109.1 SNMPv2-SMI::enterprises.3032.64.4.1.2.4.101.120.105.109.1 = STRING: "130" SNMP OK - "130" | Surly this should return a warning. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3545799&group_id=29880 From noreply at sourceforge.net Thu Jul 19 11:40:06 2012 From: noreply at sourceforge.net (SourceForge.net) Date: Thu, 19 Jul 2012 02:40:06 -0700 Subject: [Nagiosplug-devel] [ nagiosplug-Bugs-3545799 ] check_snmp always returns OK Message-ID: Bugs item #3545799, was opened at 2012-07-19 02:29 Message generated for change (Comment added) made by hweiss You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3545799&group_id=29880 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Argument proccessing Group: release-1.4.15 >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Tim (tj8) Assigned to: Nobody/Anonymous (nobody) Summary: check_snmp always returns OK Initial Comment: just upgrade plugins to nagios-plugins-1.4.15_1 when i try and use the command: ./check_snmp -H testserver -o enterprises.3032.64.4.1.2.4.101.120.105.109.1 -C public -w 100 -v I get: /usr/local/bin/snmpget -t 1 -r 5 -m ALL -v 1 [authpriv] testserver:161 enterprises.3032.64.4.1.2.4.101.120.105.109.1 SNMPv2-SMI::enterprises.3032.64.4.1.2.4.101.120.105.109.1 = STRING: "130" SNMP OK - "130" | Surly this should return a warning. ---------------------------------------------------------------------- >Comment By: Holger Weiss (hweiss) Date: 2012-07-19 02:40 Message: This is fixed in the 1.4.16 release. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=397597&aid=3545799&group_id=29880 From david.bonnell at osmotion.com.au Mon Jul 23 07:04:23 2012 From: david.bonnell at osmotion.com.au (Dave Bonnell) Date: Mon, 23 Jul 2012 05:04:23 +0000 (UTC) Subject: [Nagiosplug-devel] =?utf-8?q?1=2E4=2E16=3A_Error_in_check=5Fhttp?= =?utf-8?q?=2Ec_when_compiling=09without_SSL?= References: Message-ID: Michael LaCorte writes: > > Hello, > I was compiling the latest release of Nagios Plugins when I ran into a compilation issue. Make was returning the following error when trying to compile check_http.c (v1.4.16): > > ? ?check_http.c:312:9: error: ?ssl_version? undeclared (first use in this function) > > After looking at the file, I noticed that 'ssl_version' variable was declared within an '#ifdef HAVE_SSL'. After moving the 'ssl_version' declaration outside of the '#ifdef', it compiled fine. I couldn't find anything about this on Google, so I'm sending you this email to make sure you are aware of the issue. > > Thanks, > Michael LaCorte Defining the variable only when HAVE_SSL is declared sounds correct. The real issue is appears to be the reference to ssl_version at line# 312. I believe the #endif at line# 309 should be #else and the #endif should be moved to just before the break at line# 320. BTW, if you want SSL support, install libssl-dev and re-run configure. Make will then succeed ;-) Regards, Dave From mlacorte365 at gmail.com Mon Jul 23 07:20:32 2012 From: mlacorte365 at gmail.com (Michael LaCorte) Date: Sun, 22 Jul 2012 22:20:32 -0700 Subject: [Nagiosplug-devel] 1.4.16: Error in check_http.c when compiling without SSL In-Reply-To: References: Message-ID: <0957CCC9-6B31-4081-ADF1-D04F00212EC4@gmail.com> Thanks for the response. I later did install SSL since I wanted to install NRPE anyway. Declaring 'ssl_version' outside the HAVE_SSL was just a workaround that I used to get the Makefile to build. Ideally you shouldn't need to check the SSL version if it won't be compiling with SSL, but that would have been more work for me, and I just wanted to get the plugins to install. On Jul 22, 2012, at 10:04 PM, Dave Bonnell wrote: > Michael LaCorte writes: > >> >> Hello, >> I was compiling the latest release of Nagios Plugins when I ran into a > compilation issue. Make was returning the following error when trying to compile > check_http.c (v1.4.16): >> >> check_http.c:312:9: error: ?ssl_version? undeclared (first use in this > function) >> >> After looking at the file, I noticed that 'ssl_version' variable was declared > within an '#ifdef HAVE_SSL'. After moving the 'ssl_version' declaration outside > of the '#ifdef', it compiled fine. I couldn't find anything about this on > Google, so I'm sending you this email to make sure you are aware of the issue. >> >> Thanks, >> Michael LaCorte > > Defining the variable only when HAVE_SSL is declared sounds correct. The real > issue is appears to be the reference to ssl_version at line# 312. I believe the > #endif at line# 309 should be #else and the #endif should be moved to just > before the break at line# 320. > > BTW, if you want SSL support, install libssl-dev and re-run configure. Make > will then succeed ;-) > > > Regards, > > Dave > > > > ------------------------------------------------------------------------------ > Live Security Virtual Conference > Exclusive live event will cover all the ways today's security and > threat landscape has changed and how IT managers can respond. Discussions > will include endpoint security, mobile security and the latest in malware > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ > _______________________________________________________ > Nagios Plugin Development Mailing List Nagiosplug-devel at lists.sourceforge.net > Unsubscribe at https://lists.sourceforge.net/lists/listinfo/nagiosplug-devel > ::: Please include plugins version (-v) and OS when reporting any issue. > ::: Messages without supporting info will risk being sent to /dev/null From reply+i-5867840-ceae8cc4db82598d8f33c910ec7c1861340d505c-1651316 at reply.github.com Fri Jul 27 02:52:34 2012 From: reply+i-5867840-ceae8cc4db82598d8f33c910ec7c1861340d505c-1651316 at reply.github.com (William Leibzon) Date: Thu, 26 Jul 2012 17:52:34 -0700 Subject: [Nagiosplug-devel] [nagios-plugins] fixed small bug in in previous patch (#11) Message-ID: my previous certificate expiration check patch had a bug and would not have properly checked if warning threshold had numeric values, the actual threshold checking code had no problems You can merge this Pull Request by running: git pull https://github.com/willixix/nagios-plugins master Or you can view, comment on it, or merge it online at: https://github.com/nagios-plugins/nagios-plugins/pull/11 -- Commit Summary -- * applied patch that adds both critical and warning thresholds to certificate expiration checks of check_tcp, check_http, check_smtp * manually merged latest update that added SSL version parameter into my code that adds warn and crit thresholds for SSL * added test case for certificate checs with both warning and critical values * fixing bug in certificate experiation patch checking for bad warning threshold data * merging my bug fixes with master branch -- File Changes -- M plugins/check_http.c (6) M plugins/check_smtp.c (6) M plugins/check_tcp.c (2) -- Patch Links -- https://github.com/nagios-plugins/nagios-plugins/pull/11.patch https://github.com/nagios-plugins/nagios-plugins/pull/11.diff --- Reply to this email directly or view it on GitHub: https://github.com/nagios-plugins/nagios-plugins/pull/11 From reply+i-5867954-c348a0f4201a44527ec98b959e12ffabca4fcf82-1651316 at reply.github.com Fri Jul 27 03:01:35 2012 From: reply+i-5867954-c348a0f4201a44527ec98b959e12ffabca4fcf82-1651316 at reply.github.com (William Leibzon) Date: Thu, 26 Jul 2012 18:01:35 -0700 Subject: [Nagiosplug-devel] [nagios-plugins] fixed small bug in previous applied certificate expiration patch (#12) Message-ID: my previous certificate expiration check patch had a bug and would not have properly checked if warning threshold had numeric values, the actual threshold checking code had no problems You can merge this Pull Request by running: git pull https://github.com/willixix/nagios-plugins master Or you can view, comment on it, or merge it online at: https://github.com/nagios-plugins/nagios-plugins/pull/12 -- Commit Summary -- * applied patch that adds both critical and warning thresholds to certificate expiration checks of check_tcp, check_http, check_smtp * manually merged latest update that added SSL version parameter into my code that adds warn and crit thresholds for SSL * added test case for certificate checs with both warning and critical values * fixing bug in certificate experiation patch checking for bad warning threshold data * merging my bug fixes with master branch * fixing usage function corrupted when merged -- File Changes -- M plugins/check_http.c (6) M plugins/check_smtp.c (2) M plugins/check_tcp.c (2) -- Patch Links -- https://github.com/nagios-plugins/nagios-plugins/pull/12.patch https://github.com/nagios-plugins/nagios-plugins/pull/12.diff --- Reply to this email directly or view it on GitHub: https://github.com/nagios-plugins/nagios-plugins/pull/12 From reply+i-5868003-8bc66ce72e88ea16e48c0b6efcef4e9decaa75c0-1651316 at reply.github.com Fri Jul 27 03:05:30 2012 From: reply+i-5868003-8bc66ce72e88ea16e48c0b6efcef4e9decaa75c0-1651316 at reply.github.com (William Leibzon) Date: Thu, 26 Jul 2012 18:05:30 -0700 Subject: [Nagiosplug-devel] [nagios-plugins] fixed small bug in previous applied certificate expiration patch (#13) Message-ID: my previous certificate expiration check patch had a bug and would not have properly checked if warning threshold had numeric values, the actual threshold checking code had no problems You can merge this Pull Request by running: git pull https://github.com/willixix/nagios-plugins master Or you can view, comment on it, or merge it online at: https://github.com/nagios-plugins/nagios-plugins/pull/13 -- Commit Summary -- * applied patch that adds both critical and warning thresholds to certificate expiration checks of check_tcp, check_http, check_smtp * manually merged latest update that added SSL version parameter into my code that adds warn and crit thresholds for SSL * added test case for certificate checs with both warning and critical values * fixing bug in certificate experiation patch checking for bad warning threshold data * merging my bug fixes with master branch * fixing usage function corrupted when merged * another merge error I not noticed -- File Changes -- M plugins/check_http.c (2) M plugins/check_smtp.c (2) M plugins/check_tcp.c (2) -- Patch Links -- https://github.com/nagios-plugins/nagios-plugins/pull/13.patch https://github.com/nagios-plugins/nagios-plugins/pull/13.diff --- Reply to this email directly or view it on GitHub: https://github.com/nagios-plugins/nagios-plugins/pull/13