summaryrefslogtreecommitdiffstats
path: root/lib/Monitoring/Plugin/Getopt.pm
AgeCommit message (Collapse)AuthorFilesLines
2017-11-15Add missing `label` key to named Getopts argsTom Ryder1-1/+1
If the `label` key is given as a named argument for argument specification, if it turns out to be the first element returned in the argument list subject to hash randomization, then this test for named arguments fails due to the omission of the label from this alternating group in the pattern. When this happens, the code for an unnamed list of items was run instead, leading to a very confusing area happening randomly: 8 parameters were passed to Monitoring::Plugin::Getopt::arg but 2 - 5 were expected at PERL/lib/perl5/Monitoring/Plugin/Getopt.pm line 397. Monitoring::Plugin::Getopt::arg(undef, "label", "HOSTNAME", "required", 1, "help", "Hostname of device to check", "spec", ...) called at PERL/lib/perl5/Monitoring/Plugin.pm line 161 Monitoring::Plugin::add_arg(Monitoring::Plugin=HASH(0x1f90fd8), "label", "HOSTNAME", "required", 1, "help", "Hostname of device to check", "spec", ...) called at libexec/check_example line 144 If you specified all five keys for your argument, then this happens (roughly) one-fifth of the time.
2016-11-19Allow negation of command line arguments using '--no'-prefixManfred Stock1-4/+8
Getopt::Long supports negatable boolean options by appending an '!' to the option specification, so this allows to use this functionality with Monitoring::Plugin::Getopt as well.
2016-06-22use case insensitive regexSven Nierlein1-1/+1
2016-04-20Update Getopt.pmPaul Dugas1-1/+1
Allow hypen or underscore in plugin name.
2016-04-20Update Getopt.pmPaul Dugas1-1/+1
Wrong case in ALRM regex.
2015-04-11fix "Redundant argument in sprintf" in perl 5.21 (RT #103214)Sven Nierlein1-7/+9
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2014-12-28Merge pull request #2 from evgeni/getopt_colon_specSven Nierlein1-4/+14
GetOpt::Long optional arguments using a colon instead of an equal sign
2014-12-08fix link to documentationSven Nierlein1-1/+1
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2014-12-08correct getopt helpSven Nierlein1-2/+2
its not true that the options are available via the main Monitoring::Plugin object. Instead you have to fetch them from the opts object. Signed-off-by: Sven Nierlein <sven@nierlein.de>
2014-10-03format optional arguments in square braketsEvgeni Golov1-3/+13
2014-10-03GetOpt::Long optional arguments using a colon instead of an equal signEvgeni Golov1-2/+2
Instead of writing `foo|f=s` you can also write `foo|f:s` for a GetOpt::Long option spec [1], thus making the argument optional. The current implementation of `_spec_to_help` will wrongly render this as two long options: --dirport, --d:9030 directory port instead of a short and a long one: -d, --dirport=INTEGER directory port This commit fixes the the parsing of the spec, detection of the type and adds tests for a few common cases this could be used in. [1] http://perldoc.perl.org/Getopt/Long.html#Summary-of-Option-Specifications
2014-01-20keep the old name in copyright for more transparencyv0.37Sven Nierlein1-4/+8
Signed-off-by: Sven Nierlein <sven@nierlein.de>
2014-01-20renamed module into Monitoring::PluginSven Nierlein1-0/+869
since the complete monitoring team has been renamed, we also rename this module. Signed-off-by: Sven Nierlein <sven@nierlein.de>