On 07/19/2012 03:07 PM, Rob Crittenden wrote:
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
Okay. Still it makes sense to do the right thing, DN(x) instead of
Freeipa-devel mailing list