Issue #861 has been updated by Clément OUDOT.

Category set to Core
Status changed from New to Assigned
Assigned to set to Raphaël Ouazana
Target version set to 2.2

Yes seems not useful to return these attributes. Thanks for the patch.
----------------------------------------
Bug #861: Get all entries request unecessary attributes
http://tools.lsc-project.org/issues/861

Author: Frédéric POISSON
Status: Assigned
Priority: Low
Assigned to: Raphaël Ouazana
Category: Core
Target version: 2.2
Problem in version: 


Hello,

I see that on an ldap to ldap configuration that the first get all entries 
request some Java attributes which are "objectClass javaSerializedData 
javaClassName javaFactory javaCodeBase javaReferenceAddress javaClassNames 
javaRemoteLocation". 

While lookgin to the 2.1.3 source code i see that doGetAttrsList method from 
JndiServices.java initialize a SearchControl with setReturningObjFlag(true) or 
this code add the following attributes to the pivotAttributes attributes list, 
and which correspond to the com.sun.jndi.ldap.Obj JAVA_ATTRIBUTES :

65     // LDAP attributes used to support Java objects.
66     static final String[] JAVA_ATTRIBUTES = {
67         "objectClass",
68         "javaSerializedData",
69         "javaClassName",
70         "javaFactory",
71         "javaCodeBase",
72         "javaReferenceAddress",
73         "javaClassNames",
74         "javaRemoteLocation"     // Deprecated
75     };

Is it possible to apply the following patch inside JndiServices.java ? Or is it 
a wanted behavior ?

<pre>
Index: JndiServices.java
===================================================================
--- JndiServices.java   (revision 2053)
+++ JndiServices.java   (working copy)
@@ -1105,7 +1105,7 @@
                constraints.setDerefLinkFlag(false);
                constraints.setReturningAttributes(attributes);
                constraints.setSearchScope(scope);
-               constraints.setReturningObjFlag(true);
+               //constraints.setReturningObjFlag(true);
 
                try {
                        boolean requestPagedResults = false;
</pre>


-- 
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