From a1a7ecdc7bf6686adf8558cedd3964f9e4805469 Mon Sep 17 00:00:00 2001 From: Martin Babinsky Date: Tue, 12 Jul 2016 13:44:49 +0200 Subject: [PATCH] vault-add: set the default vault type on the client side if none was given `vault-add` commands does much processing depending on the vault type even before the request is forwarded to remote server. Since default values for parameters are now filled only on server side, the client-side logic would fail if the vault type was not explicitly given. In this case we have to retrieve and use the default vault type from schema. https://fedorahosted.org/freeipa/ticket/6047 Reviewed-By: Stanislav Laznicka --- ipaclient/plugins/vault.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/ipaclient/plugins/vault.py b/ipaclient/plugins/vault.py index bf0242fc4..a3ce6fecb 100644 --- a/ipaclient/plugins/vault.py +++ b/ipaclient/plugins/vault.py @@ -221,6 +221,11 @@ class vault_add(Local): def forward(self, *args, **options): vault_type = options.get('ipavaulttype') + + if vault_type is None: + internal_cmd = self.api.Command.vault_add_internal + vault_type = internal_cmd.params.ipavaulttype.default + password = options.get('password') password_file = options.get('password_file') public_key = options.get('ipavaultpublickey')