Issue #4331 Async Close Complete

WIP - moved the outstate from HttpOutput to HttpChannelState Error handling needs a big review Signed-off-by: Greg Wilkins <gregw@webtide.com>
jetty · Nov 30, 2019 · 2b5ffeb · 2b5ffeb
1 parent 2fe92fc
commit 2b5ffeb
Show file tree

Hide file tree

Showing 8 changed files with 538 additions and 572 deletions.
diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannel.java b/jetty-server/src/main/java/org/eclipse/jetty/server/HttpChannel.java
@@ -717,6 +717,13 @@ public boolean onRequestComplete()
         return result;
     }
 
+    void onResponseComplete()
+    {
+        if (LOG.isDebugEnabled())
+            LOG.debug("onResponseComplete {}", this);
+        _combinedListener.onResponseEnd(_request);
+    }
+
     public void onCompleted()
     {
         if (LOG.isDebugEnabled())
@@ -1194,8 +1201,6 @@ public void succeeded()
                 _combinedListener.onResponseCommit(_request);
             if (_length > 0)
                 _combinedListener.onResponseContent(_request, _content);
-            if (_complete && _state.completeResponse())
-                _combinedListener.onResponseEnd(_request);
             super.succeeded();
         }