From e415da22f350fbda5b8b341bf2dc5f969cecb84a Mon Sep 17 00:00:00 2001 From: David Kupka Date: May 16 2017 09:39:43 +0000 Subject: otptoken-add-yubikey: When --digits not provided use default value Since Thin client was introduced default values for options are not populated in client side plugins. When option has default value and is needed in client plugin it must be handled by explicitly. https://pagure.io/freeipa/issue/6900 Reviewed-By: Stanislav Laznicka --- diff --git a/ipaclient/plugins/otptoken_yubikey.py b/ipaclient/plugins/otptoken_yubikey.py index 9993ec8..60b73a7 100644 --- a/ipaclient/plugins/otptoken_yubikey.py +++ b/ipaclient/plugins/otptoken_yubikey.py @@ -147,7 +147,10 @@ class otptoken_add_yubikey(Command): # Write the config. cfg = yk.init_config() - cfg.mode_oath_hotp(key, kwargs['ipatokenotpdigits']) + cfg.mode_oath_hotp(key, kwargs.get( + 'ipatokenotpdigits', + self.get_default_of('ipatokenotpdigits') + )) cfg.extended_flag('SERIAL_API_VISIBLE', True) yk.write_config(cfg, slot=kwargs['slot'])