2011/10/20 <[email protected]>:
> Author: markt
> Date: Wed Oct 19 20:55:39 2011
> New Revision: 1186485
>
> URL: http://svn.apache.org/viewvc?rev=1186485&view=rev
> Log:
> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=52055
> Buffers not correctly reset between keep-alive requests when using async
>
> Modified:
> tomcat/tc7.0.x/trunk/ (props changed)
>
> tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java
> tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
>
> ---
> tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java
> (original)
> +++
> tomcat/tc7.0.x/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java
> Wed Oct 19 20:55:39 2011
> @@ -1538,6 +1538,8 @@ public abstract class AbstractHttp11Proc
> if (!keepAlive) {
> return SocketState.CLOSED;
> } else {
> + getInputBuffer().nextRequest();
> + getOutputBuffer().nextRequest();
> return SocketState.OPEN;
> }
> }
>
Searching for places that use org.apache.coyote.Constants.STAGE_ENDED ...
1) it looks like AbstractAjpProcessor#asyncDispatch(SocketStatus)
needs similar patch:
it has "return SocketState.OPEN;"
If I understand correctly in Ajp that will be recycle(false)
2) In Http11NioProcessor#event() I do not see nextRequest() calls, but
similar implementation in Http11AprProcessor#event() has nextRequest()
calls.
Best regards,
Konstantin Kolinko
> --- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
> +++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Wed Oct 19 20:55:39 2011
> @@ -110,6 +110,11 @@
> authentication from being lost during the authentication process.
> (markt)
> </fix>
> + <fix>
> + <bug>52055</bug>: Ensure that the input and output buffers are
> correctly
> + reset between keep-alive requests when using Servlet 3.0 asynchronous
> + request processing. (markt)
> + </fix>
> </changelog>
> </subsection>
> <subsection name="Coyote">
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
