This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/8.5.x by this push: new 923d834 Ensure HTTP/1.1 processor is recycled after a direct h2c connection 923d834 is described below commit 923d834500802a61779318911d7898bd85fc950e 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 39153f1..577ebbf 100644 --- a/java/org/apache/coyote/AbstractProtocol.java +++ b/java/org/apache/coyote/AbstractProtocol.java @@ -826,8 +826,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 connections.put(socket, processor); @@ -837,7 +839,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 15f36ae..a46bbca 100644 --- a/webapps/docs/changelog.xml +++ b/webapps/docs/changelog.xml @@ -103,6 +103,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="Other"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org