remove commented code

caching will be in LdapRbacManager.

git-svn-id: https://svn.apache.org/repos/asf/archiva/redback/redback-core/trunk@1428613 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/DefaultLdapRoleMapper.java b/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/DefaultLdapRoleMapper.java
index 4df4bc731672ec0208abf23c82a1ef1cef1a9e1b..466ad2ed3066b0e57b1b3e66dc3d44f648018731 100644 (file)
--- a/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/DefaultLdapRoleMapper.java
+++ b/redback-rbac/redback-rbac-providers/redback-rbac-ldap/src/main/java/org/apache/archiva/redback/rbac/ldap/DefaultLdapRoleMapper.java
@@ -89,7 +89,6 @@ public class DefaultLdapRoleMapper
     public List<String> getAllGroups()
         throws MappingException
     {
-        // TODO caching
         LdapConnection ldapConnection = null;
 
         NamingEnumeration<SearchResult> namingEnumeration = null;
@@ -158,7 +157,6 @@ public class DefaultLdapRoleMapper
     public List<String> getGroupsMember( String group )
         throws MappingException
     {
-        // TODO caching
         LdapConnection ldapConnection = null;
 
         NamingEnumeration<SearchResult> namingEnumeration = null;
@@ -228,20 +226,6 @@ public class DefaultLdapRoleMapper
     public List<String> getGroups( String username )
         throws MappingException
     {
-        // TODO caching and a filter with uid
-
-        /*List<String> allGroups = getAllGroups();
-        List<String> userGroups = new ArrayList<String>();
-        for ( String group : allGroups )
-        {
-            List<String> users = getGroupsMember( group );
-            if ( users.contains( username ) )
-            {
-                userGroups.add( group );
-            }
-        }
-        return userGroups;
-        */
 
         List<String> userGroups = new ArrayList<String>();
 
@@ -285,8 +269,6 @@ public class DefaultLdapRoleMapper
                         // uid=blabla we only want bla bla
                         userName = StringUtils.substringAfter( userName, "=" );
                         userName = StringUtils.substringBefore( userName, "," );
-                        //log.debug( "found group for username {}: '{}", group, userName );
-
                         allMembers.add( userName );
                     }
                     close( allMembersEnum );