You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2013/09/03 16:59:18 UTC
svn commit: r1519711 -
/tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java
Author: markt
Date: Tue Sep 3 14:59:17 2013
New Revision: 1519711
URL: http://svn.apache.org/r1519711
Log:
Fixing the last Gump failure, created this one.
Rethrow the exception so that the access log entry is created.
Error handling should be consistent across the connectors now.
Modified:
tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java
Modified: tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java?rev=1519711&r1=1519710&r2=1519711&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java (original)
+++ tomcat/trunk/java/org/apache/catalina/connector/CoyoteAdapter.java Tue Sep 3 14:59:17 2013
@@ -366,7 +366,7 @@ public class CoyoteAdapter implements Ad
} catch (Throwable t) {
ExceptionUtils.handleThrowable(t);
res.getWriteListener().onError(t);
- return false;
+ throw t;
} finally {
Thread.currentThread().setContextClassLoader(oldCL);
}
@@ -386,7 +386,7 @@ public class CoyoteAdapter implements Ad
} catch (Throwable t) {
ExceptionUtils.handleThrowable(t);
req.getReadListener().onError(t);
- return false;
+ throw t;
} finally {
Thread.currentThread().setContextClassLoader(oldCL);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org