From: Brett Porter <brett@apache.org>
Date: Sun, 8 Mar 2009 20:45:02 +0000 (+0000)
Subject: [MRM-1109] environment checks do not indicate which caused a violation
X-Git-Tag: archiva-1.2~91
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3a742eeb67e72d3624c53d9e1f9f950fbbd836c3;p=archiva.git

[MRM-1109] environment checks do not indicate which caused a violation


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

diff --git a/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java b/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java
index d03612ca1..76ca70288 100644
--- a/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java
+++ b/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java
@@ -146,8 +146,13 @@ public class SecuritySynchronization
         for ( Entry<String, EnvironmentCheck> entry : checkers.entrySet() )
         {
             EnvironmentCheck check = entry.getValue();
-            log.info( "Running Environment Check: " + entry.getKey() );
-            check.validateEnvironment( violations );
+            List<String> v = new ArrayList<String>();
+            check.validateEnvironment( v );
+            log.info( "Environment Check: " + entry.getKey() + " -> " + v.size() + " violation(s)" );
+            for ( String s : v )
+            {
+                violations.add( "[" + entry.getKey() + "] " + s );
+            }
         }
 
         if ( CollectionUtils.isNotEmpty( violations ) )