On 06/29/2016 07:25 AM, Fraser Tweedale wrote:
The attached patch fixes
https://fedorahosted.org/freeipa/ticket/5991.
Thanks,
Fraser
Hi Fraser,
A few cosmetic comments:
PEP8 issues:
./ipalib/errors.py:1399:1: E302 expected 2 blank lines, found 1
./ipaserver/plugins/cert.py:394:80: E501 line too long (98 > 79 characters)
./ipaserver/plugins/cert.py:496:80: E501 line too long (81 > 79 characters)
and there is a typo in ipaserver/plugins/cert.py
+ doc=_("automatically add the principal if it doesn't exist
(service princpals only)"),
should be "princ*i*pals only"
Otherwise LGTM,
Flo
--
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