You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@sling.apache.org by fm...@apache.org on 2010/10/02 01:02:45 UTC

svn commit: r1003700 - /sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingRequestProcessorImpl.java

Author: fmeschbe
Date: Fri Oct  1 23:02:45 2010
New Revision: 1003700

URL: http://svn.apache.org/viewvc?rev=1003700&view=rev
Log:
SLING-1815 Don't let IOException be handled by the Throwable catch-all but instead rethrow it to be handled by the caller as declared in the method signature

Modified:
    sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingRequestProcessorImpl.java

Modified: sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingRequestProcessorImpl.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingRequestProcessorImpl.java?rev=1003700&r1=1003699&r2=1003700&view=diff
==============================================================================
--- sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingRequestProcessorImpl.java (original)
+++ sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingRequestProcessorImpl.java Fri Oct  1 23:02:45 2010
@@ -205,6 +205,11 @@ public class SlingRequestProcessorImpl i
             log.error("{} , sending status {}", errorMessage, status);
             servletResponse.sendError(status, errorMessage);
 
+        } catch (IOException ioe) {
+
+            // forward IOException up the call chain to properly handle it
+            throw ioe;
+
         } catch (Throwable t) {
 
             // if we have request data and a non-null active servlet name