Robert Crittenden wrote:
This looks good, I just want to amend the commit with the tickets that this 
addresses before pushing.

It seems that this covers 1670-1674. Is 1392 included as well?


----- Original Message -----
From: "John Dennis" <>
To: "freeipa-devel" <>
Sent: Friday, August 10, 2012 1:11:23 PM
Subject: [Freeipa-devel] [PATCH 76] Use DN objects instead of strings

Due to the size and disruptive nature of this patch it was reviewed and
tested outside the normal patch review process, see the thread "DN patch
and documentation" as well as other threads on this list. This patch is
the cumulative result of that review process.


Posting this since I haven't pushed the patch yet. I found this installing with --selfsign:

diff --git a/ipaserver/install/ b/ipaserver/install/
index 1bbee35..eebaa48 100644
--- a/ipaserver/install/
+++ b/ipaserver/install/
@@ -1048,7 +1048,7 @@ class CertDB(object):
         # Prepare a simple cert request
         req_dict = dict(PASSWORD=self.gen_password(),
-                        CERTNAME=DN('CN', nickname))
+                        CERTNAME=DN(('CN', nickname)))
         req_template = ipautil.SHARE_DIR + reqcfg + ".template"
         conf = ipautil.template_file(req_template, req_dict)
         fd = open(reqcfg, "w+")

Freeipa-devel mailing list

Reply via email to