Petr Viktorin wrote:
On 07/17/2012 08:52 PM, Rob Crittenden wrote:
When determining whether a member is direct or indirect we were not
doing a case-insensitive comparison which led to marking a member as
both direct and indirect (in a test case no less).
This patch fixes the comparison and the test.
rob
When comparing DNs, you should use the DN class, not string
manipulations: DN(x) instead of x.lower().
How urgent is this? John's DN patch solves this in a much more thorough
way, maybe it'd be better to just wait for that.
The problem is we're currently reporting incorrect membership. I figured
this would be a short-term fix unless you think the DN patch commit is
imminent.
rob
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
