diff options
author | Stanislav Laznicka <slaznick@redhat.com> | 2017-02-21 10:16:46 +0100 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2017-02-21 15:31:54 +0100 |
commit | 0fffeabe0249d9c3c11e522fccf22ddeb1197b64 (patch) | |
tree | cb18147b98b3c17d48e4a6c9f6f80131fba4b8fa /ipaclient | |
parent | 9ac068ad04a2323192f9447986a3d1c5431f1e50 (diff) | |
download | freeipa-0fffeabe0249d9c3c11e522fccf22ddeb1197b64.tar.gz freeipa-0fffeabe0249d9c3c11e522fccf22ddeb1197b64.tar.xz freeipa-0fffeabe0249d9c3c11e522fccf22ddeb1197b64.zip |
Fix error in ca_cert_files validator
ClientInstall expects a single ca_cert_file as a string but the
framework gives it a list.
https://fedorahosted.org/freeipa/ticket/6694
Reviewed-By: Christian Heimes <cheimes@redhat.com>
Diffstat (limited to 'ipaclient')
-rw-r--r-- | ipaclient/install/client.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/ipaclient/install/client.py b/ipaclient/install/client.py index f951770e5..774eaaf5b 100644 --- a/ipaclient/install/client.py +++ b/ipaclient/install/client.py @@ -3555,6 +3555,10 @@ class ClientInstall(ClientInstallInterface, @ca_cert_files.validator def ca_cert_files(self, value): + if not isinstance(value, list): + raise ValueError("Expected list, got {!r}".format(value)) + # this is what init() does + value = value[-1] if not os.path.exists(value): raise ValueError("'%s' does not exist" % value) if not os.path.isfile(value): |