You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by ol...@apache.org on 2013/01/16 16:16:28 UTC

svn commit: r1433977 - /archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java

Author: olamy
Date: Wed Jan 16 15:16:28 2013
New Revision: 1433977

URL: http://svn.apache.org/viewvc?rev=1433977&view=rev
Log:
catch NameNotFoundException to return false as we check a group existence

Modified:
    archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java

Modified: archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java
URL: http://svn.apache.org/viewvc/archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java?rev=1433977&r1=1433976&r2=1433977&view=diff
==============================================================================
--- archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java (original)
+++ archiva/redback/redback-core/trunk/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java Wed Jan 16 15:16:28 2013
@@ -33,6 +33,7 @@ import org.springframework.stereotype.Se
 import javax.annotation.PostConstruct;
 import javax.inject.Inject;
 import javax.inject.Named;
+import javax.naming.NameNotFoundException;
 import javax.naming.NamingEnumeration;
 import javax.naming.NamingException;
 import javax.naming.directory.Attribute;
@@ -205,6 +206,11 @@ public class DefaultLdapRoleMapper
 
             return namingEnumeration.hasMore();
         }
+        catch ( NameNotFoundException e )
+        {
+            log.debug( "group {} for role {} not found", groupName, roleName );
+            return false;
+        }
         catch ( LdapException e )
         {
             throw new MappingException( e.getMessage(), e );