LDAP-46: DirContextAdapter.getModificatioItems() doesnt ignore case when collecting values to be removed and added.

[spring-projects-issues]

LDAP-46: DirContextAdapter.getModificatioItems() doesnt ignore case when collecting values to be removed and added.

[spring-projects-issues]

Mattias Hellborg Arthursson said:

This issue should be addressed, but I wouldn’t want to do a hack just to make string comparisons be case-insensitive. First of all, while it is the most common case, not all Attributes are case-insensitive. Secondly, the same problem (or at least a similar one) applies to other Attributes as well. E.g. ‘uniqueMember’ represents a DN, but the same DN can be represented by different strings (i.e. with spaces etc.), so the same problem would apply there as well (if the Directory Server is grumpy).

We should address these types of problems in a generic and configurable way. Configuration won’t be too easy, the main problem being that DirContextAdapter instances are created all over the place (from DefaultDirObjectFactory as well as by users in their Dao methods). Bottom line, we’ll need to think more about this to get it done properly.

[spring-projects-issues]

Mattias Hellborg Arthursson said:

Maybe we don’t need a complicated generic solution to this after all. An idea came up on the forum (http://forum.springframework.org/showthread.php?p=113359#post113359), which proposes to just make sure that the REMOVE_ATTRIBUTE items come first. While this will change Attribute value that really didn’t syntactically change, that’s actually not such a bad thing – after all what will be stored is the value in the case or format that the user actually specified.

[spring-projects-issues]

Mattias Hellborg Arthursson said:

Did a fix for this – it is available in the nightly snapshots (as of tomorrow). Haven’t been able to verify it though, as the problem doesn’t present itself on ApacheDS (where we run our integration tests. I would appreciate it if you could verify it on your platform.

[spring-projects-issues]

Mattias Hellborg Arthursson said:

Btw, the nightly snapshots are here:
http://static.springframework.org/downloads/nightly/spring-ldap.php

[spring-projects-issues]

Mattias Hellborg Arthursson said:

Reworked getModificationItems() to use Attribute.contains() to determine attribute value equality. Also, removed values are now always returned first in the ModificationItem array, avoiding the ATT_ALREAY_EXIST problems even if the Attribute implementation should be unable to determine syntactically correct Attribute equality.

In the future we might look further into this and use getAttrubuteSyntaxDefinition/getAttributeDefinition, but for the time being this will have to do.