Author: brett
Date: Tue Jul 29 09:59:43 2008
New Revision: 680769

URL: http://svn.apache.org/viewvc?rev=680769&view=rev
Log:
improve logging for remote failure

Modified:
    
archiva/trunk/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java

Modified: 
archiva/trunk/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java?rev=680769&r1=680768&r2=680769&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java
 Tue Jul 29 09:59:43 2008
@@ -822,9 +822,15 @@
         }
         catch ( WagonException e )
         {
-            throw new ProxyException(
-                "Download failure on resource [" + remoteRepository.getURL() + 
"/" + remotePath + "]:" + e.getMessage(),
-                e );
+            // TODO: shouldn't have to drill into the cause, but 
TransferFailedException is often not descriptive enough
+            
+            String msg =
+                "Download failure on resource [" + remoteRepository.getURL() + 
"/" + remotePath + "]:" + e.getMessage();
+            if ( e.getCause() != null )
+            {
+                msg += " (cause: " + e.getCause() + ")";
+            }
+            throw new ProxyException( msg, e );
         }
         finally
         {


Reply via email to