[Freeipa-devel] [PATCH] user provided certs

Karl MacMillan kmacmill at redhat.com
Wed Dec 12 16:08:13 UTC 2007


On Wed, 2007-12-12 at 11:05 -0500, Rob Crittenden wrote:
> Karl MacMillan wrote:
> > Add a utility to import user provided certs in pkcs#12 format.
> > 
> > 
> > ------------------------------------------------------------------------
> > 
> > _______________________________________________
> > Freeipa-devel mailing list
> > Freeipa-devel at redhat.com
> > https://www.redhat.com/mailman/listinfo/freeipa-devel
> 
> You are missing a close after the write in set_http_cert_name()
> 
> I'm guessing that python automatically closes it when fd goes out of 
> scope but still...
> 

It does, but I'll fix before pushing.

> I'm not sure why you don't use installutils.update_file() to replace 
> file.append('NSSNickname').
> 

That can't properly handle cert nicknames with spaces.

Karl




More information about the Freeipa-devel mailing list