diff --git a/archiva-modules/archiva-web/archiva-security/pom.xml b/archiva-modules/archiva-web/archiva-security/pom.xml
index 510fda073..512397898 100644
--- a/archiva-modules/archiva-web/archiva-security/pom.xml
+++ b/archiva-modules/archiva-web/archiva-security/pom.xml
@@ -53,6 +53,10 @@
org.codehaus.redback
redback-rbac-role-manager
+
+ com.google.guava
+ guava
+
org.codehaus.redback
@@ -84,10 +88,6 @@
derby
test
-
- com.google.guava
- guava
- test
-
+
diff --git a/archiva-modules/archiva-web/archiva-security/src/main/java/org/apache/maven/archiva/security/DefaultUserRepositories.java b/archiva-modules/archiva-web/archiva-security/src/main/java/org/apache/maven/archiva/security/DefaultUserRepositories.java
index 88a28e2ac..c4603e411 100644
--- a/archiva-modules/archiva-web/archiva-security/src/main/java/org/apache/maven/archiva/security/DefaultUserRepositories.java
+++ b/archiva-modules/archiva-web/archiva-security/src/main/java/org/apache/maven/archiva/security/DefaultUserRepositories.java
@@ -19,9 +19,7 @@ package org.apache.maven.archiva.security;
* under the License.
*/
-import java.util.ArrayList;
-import java.util.List;
-
+import com.google.common.collect.Lists;
import org.apache.maven.archiva.configuration.ArchivaConfiguration;
import org.apache.maven.archiva.configuration.ManagedRepositoryConfiguration;
import org.codehaus.plexus.redback.authentication.AuthenticationResult;
@@ -38,14 +36,16 @@ import org.slf4j.LoggerFactory;
import org.springframework.stereotype.Service;
import javax.inject.Inject;
+import java.util.ArrayList;
+import java.util.List;
/**
* DefaultUserRepositories
- *
+ *
* @version $Id$
- * plexus.component role="org.apache.maven.archiva.security.UserRepositories" role-hint="default"
+ * plexus.component role="org.apache.maven.archiva.security.UserRepositories" role-hint="default"
*/
-@Service("userRepositories")
+@Service( "userRepositories" )
public class DefaultUserRepositories
implements UserRepositories
{
@@ -66,7 +66,7 @@ public class DefaultUserRepositories
*/
@Inject
private ArchivaConfiguration archivaConfiguration;
-
+
private Logger log = LoggerFactory.getLogger( DefaultUserRepositories.class );
public List getObservableRepositoryIds( String principal )
@@ -92,8 +92,7 @@ public class DefaultUserRepositories
List repoIds = new ArrayList();
- List repos =
- archivaConfiguration.getConfiguration().getManagedRepositories();
+ List repos = archivaConfiguration.getConfiguration().getManagedRepositories();
for ( ManagedRepositoryConfiguration repo : repos )
{
@@ -108,8 +107,11 @@ public class DefaultUserRepositories
catch ( AuthorizationException e )
{
// swallow.
- log.debug( "Not authorizing '" + principal + "' for repository '" + repo.getId() + "': "
- + e.getMessage() );
+ if ( log.isDebugEnabled() )
+ {
+ log.debug( "Not authorizing '{}' for repository '{}': {}",
+ Lists.