You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@archiva.apache.org by br...@apache.org on 2009/03/08 21:45:02 UTC
svn commit: r751517 -
/archiva/trunk/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java
Author: brett
Date: Sun Mar 8 20:45:02 2009
New Revision: 751517
URL: http://svn.apache.org/viewvc?rev=751517&view=rev
Log:
[MRM-1109] environment checks do not indicate which caused a violation
Modified:
archiva/trunk/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java
Modified: archiva/trunk/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java
URL: http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java?rev=751517&r1=751516&r2=751517&view=diff
==============================================================================
--- archiva/trunk/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java (original)
+++ archiva/trunk/archiva-modules/archiva-web/archiva-webapp/src/main/java/org/apache/maven/archiva/web/startup/SecuritySynchronization.java Sun Mar 8 20:45:02 2009
@@ -146,8 +146,13 @@
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 ) )