You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@karaf.apache.org by jb...@apache.org on 2011/05/05 08:57:29 UTC

svn commit: r1099702 - /karaf/trunk/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/ldap/LDAPLoginModule.java

Author: jbonofre
Date: Thu May  5 06:57:28 2011
New Revision: 1099702

URL: http://svn.apache.org/viewvc?rev=1099702&view=rev
Log:
[KARAF-606] Allow LDAPLoginModule to supply role "DN" from LDAP group search.

Modified:
    karaf/trunk/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/ldap/LDAPLoginModule.java

Modified: karaf/trunk/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/ldap/LDAPLoginModule.java
URL: http://svn.apache.org/viewvc/karaf/trunk/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/ldap/LDAPLoginModule.java?rev=1099702&r1=1099701&r2=1099702&view=diff
==============================================================================
--- karaf/trunk/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/ldap/LDAPLoginModule.java (original)
+++ karaf/trunk/jaas/modules/src/main/java/org/apache/karaf/jaas/modules/ldap/LDAPLoginModule.java Thu May  5 06:57:28 2011
@@ -224,6 +224,7 @@ public class LDAPLoginModule extends Abs
             } else {
                 controls.setSearchScope(SearchControls.ONELEVEL_SCOPE);
             }
+            controls.setReturningAttributes(new String[]{ roleNameAttribute });
             logger.debug("Looking for the user roles in LDAP with ");
             logger.debug("  base DN: " + roleBaseDN);
             roleFilter = roleFilter.replaceAll("%u", user);