diff options
| author | Jan Cholasta <jcholast@redhat.com> | 2016-08-08 13:09:39 +0200 |
|---|---|---|
| committer | David Kupka <dkupka@redhat.com> | 2016-08-10 08:51:39 +0200 |
| commit | e9c1d21b9fec17ab13894885eb1238631ecc43e5 (patch) | |
| tree | 1bb437d0fd79385b8b028af766bf097f80ef0a18 /ipaclient/remote_plugins | |
| parent | 771dea5c6bf1cca7b3756d2f1be48f613de14ceb (diff) | |
| download | freeipa-e9c1d21b9fec17ab13894885eb1238631ecc43e5.tar.gz freeipa-e9c1d21b9fec17ab13894885eb1238631ecc43e5.tar.xz freeipa-e9c1d21b9fec17ab13894885eb1238631ecc43e5.zip | |
parameters: move the `confirm` kwarg to Param
Whether a parameter is treated like password is determined by the
`password` class attribute defined in the Param class. Whether the CLI will
asks for confirmation of a password parameter depends on the value of the
`confirm` kwarg of the Password class.
Move the `confirm` kwarg from the Password class to the Param class, so
that it can be used by any Param subclass which has the `password` class
attribute set to True.
This fixes confirmation of the --key option of otptoken-add, which is a
Bytes subclass with `password` set to True.
https://fedorahosted.org/freeipa/ticket/6174
Reviewed-By: Martin Basti <mbasti@redhat.com>
Reviewed-By: David Kupka <dkupka@redhat.com>
Diffstat (limited to 'ipaclient/remote_plugins')
| -rw-r--r-- | ipaclient/remote_plugins/schema.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipaclient/remote_plugins/schema.py b/ipaclient/remote_plugins/schema.py index a215452ea..c06d6d278 100644 --- a/ipaclient/remote_plugins/schema.py +++ b/ipaclient/remote_plugins/schema.py @@ -167,7 +167,7 @@ class _SchemaPlugin(object): elif key in ('cli_metavar', 'cli_name'): kwargs[key] = str(value) - elif key == 'confirm' and issubclass(cls, Password): + elif key == 'confirm': kwargs[key] = value elif key == 'default': default = value |