Revert a change introduced in the fix for bug 60718 that changed the status code recorded in the access log when the client dropped the connection from 200 to 500.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1797828 13f79535-47bb-0310-9956-ffa450edef68

Author: markt-asf

java/org/apache/coyote/AbstractProcessor.java

@@ -80,7 +80,10 @@ protected AbstractProcessor(Request coyoteRequest, Response coyoteResponse) {
     protected void setErrorState(ErrorState errorState, Throwable t) {
         boolean blockIo = this.errorState.isIoAllowed() && !errorState.isIoAllowed();
         this.errorState = this.errorState.getMostSevere(errorState);
-        if (response.getStatus() < 400) {
+        // Don't change the status code for IOException since that is almost
+        // certainly a client disconnect in which case it is preferable to keep
+        // the original status code http://markmail.org/message/4cxpwmxhtgnrwh7n
+        if (response.getStatus() < 400 && !(t instanceof IOException)) {
             response.setStatus(500);
         }
         if (t != null) {

webapps/docs/changelog.xml

@@ -93,6 +93,11 @@
         <bug>61120</bug>: Do not ignore path parameters when processing HTTP/2
         requests. (markt)
       </fix>
+      <fix>
+        Revert a change introduced in the fix for bug <bug>60718</bug> that
+        changed the status code recorded in the access log when the client
+        dropped the connection from 200 to 500. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Jasper">