This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/master by this push: new c9167ae Ensure HTTP/1.1 processor is recycled after a direct h2c connection c9167ae is described below commit c9167ae30f3b03b112f3d81772e3450b7d0e6a25 Author: Mark Thomas <ma...@apache.org> AuthorDate: Fri Jun 26 12:49:50 2020 +0100 Ensure HTTP/1.1 processor is recycled after a direct h2c connection --- java/org/apache/coyote/AbstractProtocol.java | 9 ++++++--- webapps/docs/changelog.xml | 4 ++++ 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/java/org/apache/coyote/AbstractProtocol.java b/java/org/apache/coyote/AbstractProtocol.java index 1190c99..abea68e 100644 --- a/java/org/apache/coyote/AbstractProtocol.java +++ b/java/org/apache/coyote/AbstractProtocol.java @@ -853,8 +853,10 @@ public abstract class AbstractProtocol<S> implements ProtocolHandler, // Assume direct HTTP/2 connection UpgradeProtocol upgradeProtocol = getProtocol().getUpgradeProtocol("h2c"); if (upgradeProtocol != null) { - processor = upgradeProtocol.getProcessor( - wrapper, getProtocol().getAdapter()); + // Release the Http11 processor to be re-used + release(processor); + // Create the upgrade processor + processor = upgradeProtocol.getProcessor(wrapper, getProtocol().getAdapter()); wrapper.unRead(leftOverInput); // Associate with the processor with the connection wrapper.setCurrentProcessor(processor); @@ -864,7 +866,8 @@ public abstract class AbstractProtocol<S> implements ProtocolHandler, "abstractConnectionHandler.negotiatedProcessor.fail", "h2c")); } - return SocketState.CLOSED; + // Exit loop and trigger appropriate clean-up + state = SocketState.CLOSED; } } else { HttpUpgradeHandler httpUpgradeHandler = upgradeToken.getHttpUpgradeHandler(); diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml index 69f9bee..46f9def 100644 --- a/webapps/docs/changelog.xml +++ b/webapps/docs/changelog.xml @@ -110,6 +110,10 @@ <fix> Reduce the memory footprint of closed HTTP/2 streams. (markt) </fix> + <fix> + Ensure that the HTTP/1.1 processor is correctly recycled when a direct + connection to h2c is made. (markt) + </fix> </changelog> </subsection> <subsection name="Jasper"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org