upstream/certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-06-04 14:26:10 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Fix detect defaults when nargs present

Browse source 
If a plugin adds a command line argument with the `nargs` parameter, e.g. `nargs='+'`, this parameter gets filtered from the `HelpfulArgumentParser` created/used in default detection. As a result, this parser does not parse the command line in the expected way, instead using the default value for `nargs`.

Leaving `nargs` in place resolves this issue.
This commit is contained in:
Scott Armitage 2018-07-07 12:16:25 -07:00 committed by Scott Armitage
parent 43f2bfd6f1
commit 15904d24c0
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
certbot/cli.py
View file

@ -120,7 +120,7 @@ More detailed help:
# These argparse parameters should be removed when detecting defaults.
ARGPARSE_PARAMS_TO_REMOVE = ("const", "nargs", "type",)
ARGPARSE_PARAMS_TO_REMOVE = ("const", "type",)
# These sets are used when to help detect options set by the user.