flo-renaud's pull request #71: "Fix regression introduced in ipa-certupdate" 
was opened

PR body:
"""
The fix for 6288 was overwritten by commit 
08b768313020c45bfa82d67cd214afabf605f4b3.

https://fedorahosted.org/freeipa/ticket/6288
"""

See the full pull-request at https://github.com/freeipa/freeipa/pull/71
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/71/head:pr71
git checkout pr71
From 6105cfed239d14a0e19926a954028c145c8d9d4b Mon Sep 17 00:00:00 2001
From: Florence Blanc-Renaud <f...@redhat.com>
Date: Fri, 9 Sep 2016 15:04:35 +0200
Subject: [PATCH] Fix regression introduced in ipa-certupdate

The fix for 6288 was overwritten by commit 08b768313020c45bfa82d67cd214afabf605f4b3.

https://fedorahosted.org/freeipa/ticket/6288
---
 ipaclient/ipa_certupdate.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/ipaclient/ipa_certupdate.py b/ipaclient/ipa_certupdate.py
index 4b97b85..52f73e1 100644
--- a/ipaclient/ipa_certupdate.py
+++ b/ipaclient/ipa_certupdate.py
@@ -83,7 +83,10 @@ def run(self):
             certs = certstore.get_ca_certs(ldap, api.env.basedn,
                                            api.env.realm, ca_enabled)
 
-            lwcas = api.Command.ca_find()['result']
+            if ca_enabled:
+                lwcas = api.Command.ca_find()['result']
+            else:
+                lwcas = []
 
             api.Backend.rpcclient.disconnect()
         finally:
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to