Issue #4331 Close Complete

Reworked order of ifs to match logic above in onWriteComplete Signed-off-by: Greg Wilkins <gregw@webtide.com>
jetty · Dec 18, 2019 · b7014a7 · b7014a7
1 parent 6872c33
commit b7014a7
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/HttpOutput.java b/jetty-server/src/main/java/org/eclipse/jetty/server/HttpOutput.java
@@ -321,24 +321,22 @@ else if (_state == State.CLOSE)
             LOG.debug("onWriteComplete({},{}) {}->{} c={} cb={} w={}",
                 last, failure, state, stateString(), BufferUtil.toDetailString(closeContent), closedCallback, wake);
 
-        if (failure != null)
-            _channel.abort(failure);
-
-        if (closeContent != null)
-        {
-            channelWrite(closeContent, true, new WriteCompleteCB());
-            return;
-        }
-
         try
         {
+            if (failure != null)
+                _channel.abort(failure);
+
             if (closedCallback != null)
             {
                 if (failure == null)
                     closedCallback.succeeded();
                 else
                     closedCallback.failed(failure);
             }
+            else if (closeContent != null)
+            {
+                channelWrite(closeContent, true, new WriteCompleteCB());
+            }
         }
         finally
         {