Hi,

Yes, it was on purpose. I have discussed a similar problem with Emmanuel few years ago. In practice, all interesting information in root DSE are operational attributes. Therefore there is almost no sense in fetching root DSE with "*" and without "+". I was fixing the code at several places back than, but I have obviously missed this one.

Sorry for that test failure. I have built the API with the usual "mvn clean install" and the build was successful, before I made a commit. I'm always forgetting that there is a second test suite for the API in the server code.

--
Radovan Semancik
Software Architect
evolveum.com

On 19. 6. 2021 16:46, Stefan Seelmann wrote:
Hi Radovan,

was this change in getRootDse() on purpose?

I noticed it because of a test failure in the server's test [1] when I
switch the API version to 2.0.3-SNAPSHOT.

But I agree with the change, it's the operational attributes that are of
interest when fetching it :-)

Kind Regards,
Stefan

[1]
https://github.com/apache/directory-server/blob/master/ldap-client-test/src/test/java/org/apache/directory/shared/client/api/operations/GetRootDseTest.java#L80


diff --git 
a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
 
b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
index 0d425a3..26c4db2 100644
--- 
a/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
+++ 
b/ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapNetworkConnection.java
@@ -4301,7 +4301,7 @@ public class LdapNetworkConnection extends 
AbstractLdapConnection implements Lda
      @Override
      public Entry getRootDse() throws LdapException
      {
-        return lookup( Dn.ROOT_DSE, SchemaConstants.ALL_USER_ATTRIBUTES_ARRAY 
);
+        return lookup( Dn.ROOT_DSE, SchemaConstants.ALL_ATTRIBUTES_ARRAY );
      }
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@directory.apache.org
For additional commands, e-mail: dev-h...@directory.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@directory.apache.org
For additional commands, e-mail: dev-h...@directory.apache.org

Reply via email to