svn commit: r1041791 - /maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/RepositoryUtils.java

[vsiveton]

Author: vsiveton
Date: Fri Dec  3 12:42:38 2010
New Revision: 1041791

URL: http://svn.apache.org/viewvc?rev=1041791&view=rev
Log:
o improved TransferFailedException catch for unknown hosts
o take care of unknown hosts to prevent undesirable ping

Modified:
    
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/RepositoryUtils.java

Modified: 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/RepositoryUtils.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/RepositoryUtils.java?rev=1041791&r1=1041790&r2=1041791&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/RepositoryUtils.java
 (original)
+++ 
maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/dependencies/RepositoryUtils.java
 Fri Dec  3 12:42:38 2010
@@ -19,6 +19,7 @@ package org.apache.maven.report.projecti
  * under the License.
  */
 
+import java.net.UnknownHostException;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -61,6 +62,8 @@ import org.codehaus.plexus.util.StringUt
  */
 public class RepositoryUtils
 {
+    private static final List<String> UNKNOWN_HOSTS = new ArrayList<String>();
+
     private final Log log;
 
     private final WagonManager wagonManager;
@@ -166,8 +169,17 @@ public class RepositoryUtils
             return false;
         }
 
+        if ( UNKNOWN_HOSTS.contains( repo.getUrl() ) )
+        {
+            if ( log.isDebugEnabled() )
+            {
+                log.debug( "The repo url '" + repo.getUrl() + "' is unknowned 
- Ignored it" );
+            }
+            return false;
+        }
+
         repo = wagonManager.getMirrorRepository( repo );
-        
+
         String id = repo.getId();
         Repository repository = new Repository( id, repo.getUrl() );
 
@@ -241,13 +253,21 @@ public class RepositoryUtils
         }
         catch ( TransferFailedException e )
         {
-            if ( log.isDebugEnabled() )
+            if ( e.getCause().getClass().isAssignableFrom( 
UnknownHostException.class ) )
             {
-                log.error( "Unable to determine if resource " + artifact + " 
exists in " + repo.getUrl(), e );
+                log.error( "Unknown host " + e.getCause().getMessage() + " - 
ignored it" );
+                UNKNOWN_HOSTS.add( repo.getUrl() );
             }
             else
             {
-                log.error( "Unable to determine if resource " + artifact + " 
exists in " + repo.getUrl() );
+                if ( log.isDebugEnabled() )
+                {
+                    log.error( "Unable to determine if resource " + artifact + 
" exists in " + repo.getUrl(), e );
+                }
+                else
+                {
+                    log.error( "Unable to determine if resource " + artifact + 
" exists in " + repo.getUrl() );
+                }
             }
             return false;
         }