search

sensu / sensu-plugin-sdk

A framework for creating Sensu plugins
MIT License
7 stars 8 forks source link

don't setup a flag unless argument string is set #37

Closed jspaleta closed 4 years ago

jspaleta commented 4 years ago

Really simple fix here

Don't setup a commandline flag when the Argument attribute in a PluginConfigOption is not set. An empty string option name leads to some weird breakages and we shouldnt allow it.

Also with this fix we can now have PluginConfigOptions that are only accessible via the keyspace+path.. which is an interesting side effect.