Hi,
In the AsyncContext.dispatch(path) javadoc we have:
"There can be at most one asynchronous dispatch operation per asynchronous
cycle, which is started by a call to one of the
ServletRequest.startAsync()<file:///C:/vily/my%20documents/servlet-3_0-final-javadoc/javax/servlet/ServletRequest.html#startAsync()>methods.
Any attempt to perform an additional asynchronous dispatch
operation within the same asynchronous cycle will result in an
IllegalStateException. If startAsync is subsequently called on the
dispatched request, then any of the dispatch or
complete()<file:///C:/vily/my%20documents/servlet-3_0-final-javadoc/javax/servlet/AsyncContext.html#complete()>methods
may be called."
If we have the following scenario:
AsyncContext asyncContext = request.startAsync(request, response);
asyncContext .dispatch("/resourceA");
asyncContext .dispatch("/resourceB");
I would assume that the second dispatch will throw ISE and a dispatching to
the "resourceA" will happen.
The current implementation (tomcat7/8) throws ISE but the dispatching is to
the "resourceB".
I prepared a change (below) that moves the check for ISE a little bit
earlier. o.a.catalina.core.AsyncContextImpl.dispatch field is not
overridden with information from the second dispatch() invocation, thus a
dispatching to "resourceA" will happen.
What do you think? Are my assumptions correct?
Thanks
Violeta
Index: C:/tc8.0.x/java/org/apache/catalina/core/AsyncContextImpl.java
===================================================================
--- C:/tc8.0.x/java/org/apache/catalina/core/AsyncContextImpl.java (revision
1486660)
+++ C:/tc8.0.x/java/org/apache/catalina/core/AsyncContextImpl.java (working
copy)
@@ -216,8 +216,8 @@
}
};
+ this.request.getCoyoteRequest().action(ActionCode.ASYNC_DISPATCH,
null);
this.dispatch = run;
- this.request.getCoyoteRequest().action(ActionCode.ASYNC_DISPATCH,
null);
}
@Override
