the real issue is adding a listener there trigger onStartAsync()*ONLY* (no onCompleted(), onTimeout(), onError() etc... cause org.apache.catalina.core.AsyncContextImpl#setStarted does listeners.clear(); )
Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber <http://www.tomitribe.com> | JavaEE Factory <https://javaeefactory-rmannibucau.rhcloud.com> 2016-05-17 12:43 GMT+02:00 Rémy Maucherat <r...@apache.org>: > 2016-05-17 12:37 GMT+02:00 Romain Manni-Bucau <rmannibu...@gmail.com>: > > > Only found https://bz.apache.org/bugzilla/show_bug.cgi?id=59261 which > > doesn't mean this code would be invalid > > > > Also if you add a listener eagerly you will have the start event and > loose > > all "end" (completed, timeout etc...) ones. This doesn't sound very > > reliable. > > > > Can you point me out the related issue please? > > > > The bugzilla title says: "Request getAsyncContext should throw > IllegalStateException if async is not started". So that's what happens now. > > Rémy >