summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Cholasta <jcholast@redhat.com>2014-09-17 15:22:19 +0200
committerPetr Viktorin <pviktori@redhat.com>2014-09-23 16:25:15 +0200
commit26188d7610170ff2fb89b12cd63a0c698a2381cb (patch)
treebf71acf38fd3b33ce6a3c4d2cf29078526680768
parentc436352e14a7d3ac4f4cf8888454c6749ddd2f13 (diff)
downloadfreeipa-26188d7610170ff2fb89b12cd63a0c698a2381cb.tar.gz
freeipa-26188d7610170ff2fb89b12cd63a0c698a2381cb.tar.xz
freeipa-26188d7610170ff2fb89b12cd63a0c698a2381cb.zip
Fix certmonger code causing the ca_renewal_master update plugin to fail
https://fedorahosted.org/freeipa/ticket/4547 Reviewed-By: David Kupka <dkupka@redhat.com>
-rw-r--r--ipapython/certmonger.py8
-rw-r--r--ipaserver/install/plugins/ca_renewal_master.py2
2 files changed, 8 insertions, 2 deletions
diff --git a/ipapython/certmonger.py b/ipapython/certmonger.py
index 62b2ba86d..b46d65b2f 100644
--- a/ipapython/certmonger.py
+++ b/ipapython/certmonger.py
@@ -166,7 +166,13 @@ def get_request_value(request_id, directive):
root_logger.error('Failed to get request: %s' % e)
raise
if request:
- return request.prop_if.Get(DBUS_CM_REQUEST_IF, directive)
+ if directive == 'ca-name':
+ ca_path = request.obj_if.get_ca()
+ ca = _cm_dbus_object(request.bus, ca_path, DBUS_CM_CA_IF,
+ DBUS_CM_IF)
+ return ca.obj_if.get_nickname()
+ else:
+ return request.prop_if.Get(DBUS_CM_REQUEST_IF, directive)
else:
return None
diff --git a/ipaserver/install/plugins/ca_renewal_master.py b/ipaserver/install/plugins/ca_renewal_master.py
index 52508b589..e24663910 100644
--- a/ipaserver/install/plugins/ca_renewal_master.py
+++ b/ipaserver/install/plugins/ca_renewal_master.py
@@ -60,7 +60,7 @@ class update_ca_renewal_master(PostUpdate):
if request_id is not None:
self.debug("found certmonger request for ipaCert")
- ca_name = certmonger.get_request_value(request_id, 'ca_name')
+ ca_name = certmonger.get_request_value(request_id, 'ca-name')
if ca_name is None:
self.warning(
"certmonger request for ipaCert is missing ca_name, "