Issue #572 has been updated by David Coutadeur.

Assigned to set to David Coutadeur
% Done changed from 0 to 90

The setMainIdentifier method is the only one setting the mainIdentifier 
attribute.
By a tests set, I examined if the value of the mainIdentifier was (or not) 
including the fullDN.
(OK if including full DN when considering an LDAP referential)

oldap-oldap (sync) (add + modify)            ... OK
oldap-oldap (async) (add + modify)           ... OK
oldap-postgre (sync) (add + modify)          ... OK
oldap-postgre (async) (add + modify)         ... OK
hsqldb-openDJ (sync) (add + modify + clean)  ... OK

I think it is sufficient to admit that the "if condition" testing the suffix 
presence is now useless.
Thus I propose the following correction to LscBean :

<pre>
    public final String getFullDistinguishedName() {
      return getMainIdentifier();
    }
</pre>
----------------------------------------
Bug #572: getFullDistinguishedName() adds a bad suffix
http://tools.lsc-project.org/issues/572

Author: David Coutadeur
Status: New
Priority: Normal
Assigned to: David Coutadeur
Category: Core
Target version: 2.0.2
Problem in version: all versions since 2.0



Whereas getDistinguishedName() is deprecated, it is the only way to do the job 
instead of getFullDistinguishedName() for the moment.
The problem comes from :

getFullDistinguishedName() 

is replacing the correct dn from :
uid=jsmith,ou=People,dc=example,dc=com
to :
uid=jsmith,ou=People,dc=example,dc=com,dc=lsc-project,dc=org


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://tools.lsc-project.org/my/account
_______________________________________________________________
Ldap Synchronization Connector (LSC) - http://lsc-project.org

lsc-dev mailing list
[email protected]
http://lists.lsc-project.org/listinfo/lsc-dev

Reply via email to