On 10/15/2014 01:08 PM, thierry bordaz wrote:
I see 2 issues with the patch:
1) Patch description should not contain "
Reviewed by:", this gets added later by a script (or human)
2) The exception handling clause should be as focused as possible, i.e. not
including whole command, but rather just the failing call, i.e.:
def post_callback(self, ldap, dn, entry, *keys, **options):
You can use
to raise the right error.
Freeipa-devel mailing list