Author: jawi
Date: Mon Oct 14 14:33:38 2013
New Revision: 1531912

URL: http://svn.apache.org/r1531912
Log:
Improved error messages by adding the failing URL; removed obsolete annotation.

Modified:
    
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/ext/impl/RemoteRepository.java
    
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/impl/RepositoryFactory.java

Modified: 
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/ext/impl/RemoteRepository.java
URL: 
http://svn.apache.org/viewvc/ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/ext/impl/RemoteRepository.java?rev=1531912&r1=1531911&r2=1531912&view=diff
==============================================================================
--- 
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/ext/impl/RemoteRepository.java
 (original)
+++ 
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/ext/impl/RemoteRepository.java
 Mon Oct 14 14:33:38 2013
@@ -83,11 +83,11 @@ public class RemoteRepository implements
         int rc = connection.getResponseCode();
         if (rc == HttpServletResponse.SC_NOT_FOUND) {
             connection.disconnect();
-            throw new IllegalArgumentException("Requested version not found in 
remote repository. (" + connection.getResponseMessage() + ")");
+            throw new IllegalArgumentException("Requested version not found in 
remote repository. (" + connection.getResponseMessage() + ") for " + 
url.toExternalForm());
         }
         else if (rc != HttpServletResponse.SC_OK) {
             connection.disconnect();
-            throw new IOException("Connection error: " + 
connection.getResponseMessage());
+            throw new IOException("Connection error: " + 
connection.getResponseMessage() + " for " + url.toExternalForm());
         }
 
         return connection.getInputStream();
@@ -130,7 +130,7 @@ public class RemoteRepository implements
                 try {
                     String line = reader.readLine();
                     if (line == null) {
-                        throw new IOException("Repository not found: 
customer=" + m_customer + ", name=" + m_name);
+                        throw new IOException("Repository not found: 
customer=" + m_customer + ", name=" + m_name + " for " + url.toExternalForm());
                     }
 
                     String representation = 
line.substring(line.lastIndexOf(','));
@@ -141,7 +141,7 @@ public class RemoteRepository implements
                 }
             }
 
-            throw new IOException("Connection error: " + 
connection.getResponseMessage());
+            throw new IOException("Connection error: " + 
connection.getResponseMessage() + " for " + url.toExternalForm());
         }
         finally {
             connection.disconnect();

Modified: 
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/impl/RepositoryFactory.java
URL: 
http://svn.apache.org/viewvc/ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/impl/RepositoryFactory.java?rev=1531912&r1=1531911&r2=1531912&view=diff
==============================================================================
--- 
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/impl/RepositoryFactory.java
 (original)
+++ 
ace/trunk/org.apache.ace.repository/src/org/apache/ace/repository/impl/RepositoryFactory.java
 Mon Oct 14 14:33:38 2013
@@ -103,7 +103,6 @@ public class RepositoryFactory implement
      *             If any of the above explanation fails <b>or</b>when there 
is an internal error creating the
      *             repository.
      */
-    @SuppressWarnings("unchecked")
     public void updated(String pid, Dictionary dict) throws 
ConfigurationException {
         String customer = (String) dict.get(REPOSITORY_CUSTOMER);
         if ((customer == null) || "".equals(customer)) {


Reply via email to