2017-06-26 16:13 GMT+03:00 Mark Thomas <ma...@apache.org>: > > On 23/06/17 14:47, Violeta Georgieva wrote: > > 2017-06-13 15:44 GMT+03:00 Mark Thomas <ma...@apache.org>: > >> On 13/06/17 11:05, Violeta Georgieva wrote: > >>> 2017-06-13 13:04 GMT+03:00 Violeta Georgieva <violet...@apache.org>: > > <snip/> > > >>>> java.lang.NullPointerException: null > >> > >> Appears to be the same cause as above, but triggered at a different point. > >> > > > > What about at least not throw NPE? > > Which object is null? Since the stack trace is from a dev version, I'm > not exactly sure where the null is in this case.
The first thread makes a dispatch and clears the request and response https://github.com/apache/tomcat/blob/trunk/java/org/apache/catalina/core/AsyncContextImpl.java#L209 the second thread handles the error and tries to set a status code https://github.com/apache/tomcat/blob/trunk/java/org/apache/catalina/core/AsyncContextImpl.java#L411 > >>>> at > >>> > > org.apache.catalina.core.AsyncContextImpl.setErrorState(AsyncContextImpl.java:411) > >>> ~[tomcat-embed-core.jar!/:8.5.16-dev] > >> > >> <snip/> > >> > >> This is consistent with what I'd expect. In both of the above cases any > >> AsyncListener.onError() methods should execute but if the app continues > >> to do things on the non-container thread then there possibility of > >> exceptions remains. > >> > > > > Ok I understand. > > > > Can we back port the change to Tomcat 8.5 so that it is guaranteed that the > > AsyncListeners will be invoked on error. > > Next on my TODO lis. > > Mark > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org > For additional commands, e-mail: dev-h...@tomcat.apache.org >
