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.


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.


Freeipa-devel mailing list

Reply via email to