On 18/02/2014 17:43, Konstantin Kolinko wrote: > 2014-02-18 20:05 GMT+04:00 Mark Thomas <ma...@apache.org>: >> On 18/02/2014 13:07, Romain Manni-Bucau wrote: >>> 2014-02-18 13:10 GMT+01:00 Mark Thomas <ma...@apache.org>: >> >> <snip/> >> >>>> Looks like there might be a couple of problems here. Any chance you >>>> could debug this and provide the full text of the merged web.xml? >>>> >>> >>> https://gist.github.com/rmannibucau/38292f66783c0d49965b >> >> OK. The error is valid. You have a 2.5 web.xml using 3.0 syntax. >> However, if the base web.xml is using 2.5, then there should not have >> been any merging. Hmm. >> >> What schema does /WEB-INF/web.xml declare in your app and where is >> TestFilter declared not to support async? >> > > In WebXml.toXml() lines 715-716 (in trunk) > [[[ > appendElement(sb, INDENT4, "async-supported", > filterDef.getAsyncSupported()); > ]]] > > needs to be wrapped by an if(version>=...). > > I tested that default conf/web.xml serializes/parses correctly but it > has no <filter>s in it, so apparently it slipped.
There are two issues here. One is the issue cited above although it is much wider than just async-supported. I'm working though the specs now and I'll extend the test cases as I go. The second issue is why tldValidation is controlling this. That I'll look at once validation is fixed. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
