On Tue, Dec 2, 2014 at 3:02 PM, Mark Thomas <ma...@apache.org> wrote:
> On 02/12/2014 22:40, Mark Kolich wrote: > > On Tue, Dec 2, 2014 at 2:04 PM, Mark Thomas <ma...@apache.org> wrote: > > > >> On 02/12/2014 20:41, Mark Kolich wrote: > >> > <snip/> > > https://issues.apache.org/bugzilla/show_bug.cgi?id=57252 is what got me > looking at this most recently. > > The tests (that I am in the process of extending) are here: > > http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/core/TestAsyncContextImpl.java?view=annotate > > This is part of the fix: > http://svn.apache.org/r1643002 > > I'm currently working on the remainder of the fix which is adjusting the > logic in the ErrorReportValve (generic error handling) and the > StandardHostValve (application defined error handling) to avoid the > concurrency issues. > > I'm close to a fix but I'm not quite there. I hope to commit the fix in > the next 24 hours or so. > Excellent. Thanks, Mark! Finally, you mentioned verifying on 9.0.x trunk -- will the fix be back ported to Tomcat 8 as well? -Mark
