----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/500/#review328 -----------------------------------------------------------
I think the entire solution is over complicated. Not a fan of introducing the processor into the input buffer, for an edge case. If you are stopping the connector, I would let the current request finish up. Since Tomcat 7 should bring the connection back to the endpoint in between keep alive requests, let the end point decide not to continue with the next request. I don't think modifying each input buffer to check if the end point is paused. Instead, I would delegate this responsibility to the endpoint to make the decision in between requests. /java/org/apache/coyote/http11/Http11NioProcessor.java <https://reviews.apache.org/r/500/#comment670> - Filip On 2011-03-13 16:05:12, markt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/500/ > ----------------------------------------------------------- > > (Updated 2011-03-13 16:05:12) > > > Review request for tomcat. > > > Summary > ------- > > Fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=50903 > Currently, if a connector is stopped with an open keep-alive connection, the > next request received on that connection will be processed. This patch > changes that so the request is not processed and the socket closed. > > > This addresses bug https://issues.apache.org/bugzilla/show_bug.cgi?id=50903. > > https://issues.apache.org/jira/browse/https://issues.apache.org/bugzilla/show_bug.cgi?id=50903 > > > Diffs > ----- > > /java/org/apache/coyote/ajp/AbstractAjpProcessor.java 1081214 > /java/org/apache/coyote/ajp/AjpAprProcessor.java 1081214 > /java/org/apache/coyote/ajp/AjpProcessor.java 1081214 > /java/org/apache/coyote/ajp/LocalStrings.properties 1081214 > /java/org/apache/coyote/http11/AbstractInputBuffer.java 1081214 > /java/org/apache/coyote/http11/Http11AprProcessor.java 1081214 > /java/org/apache/coyote/http11/Http11NioProcessor.java 1081214 > /java/org/apache/coyote/http11/Http11Processor.java 1081214 > /java/org/apache/coyote/http11/InternalAprInputBuffer.java 1081214 > /java/org/apache/coyote/http11/InternalInputBuffer.java 1081214 > /java/org/apache/coyote/http11/InternalNioInputBuffer.java 1081214 > /java/org/apache/coyote/http11/LocalStrings.properties 1081214 > /test/org/apache/catalina/connector/TestConnector.java PRE-CREATION > /test/org/apache/catalina/startup/TesterServlet.java PRE-CREATION > /test/org/apache/catalina/startup/TomcatBaseTest.java 1081214 > /test/org/apache/coyote/http11/TestTbd.java PRE-CREATION > > Diff: https://reviews.apache.org/r/500/diff > > > Testing > ------- > > Includes unit test (executed with BIO) > Manually tested with all 5 connectors. > > > Thanks, > > markt > >
