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/10/04 17:19:35 UTC

svn commit: r1529181 - /tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java

Author: markt
Date: Fri Oct  4 15:19:34 2013
New Revision: 1529181

URL: http://svn.apache.org/r1529181
Log:
Revert r1529149 as it triggers unit test failures. It needs more thought.

Modified:
    tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java

Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java?rev=1529181&r1=1529180&r2=1529181&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java (original)
+++ tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java Fri Oct  4 15:19:34 2013
@@ -1015,7 +1015,7 @@ public abstract class AbstractHttp11Proc
                     // set the status to 500 and set the errorException.
                     // If we fail here, then the response is likely already
                     // committed, so we can't try and set headers.
-                    if(!error) { // Avoid checking twice.
+                    if(keepAlive && !error) { // Avoid checking twice.
                         error = response.getErrorException() != null ||
                                 (!isAsync() &&
                                 statusDropsConnection(response.getStatus()));



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org