2014-11-18 13:29 GMT+03:00 Mark Thomas <ma...@apache.org>: > On 18/11/2014 03:48, Konstantin Kolinko wrote: >> 2014-11-18 5:30 GMT+03:00 Konstantin Kolinko <knst.koli...@gmail.com>: >>> 2014-11-17 11:06 GMT+03:00 <ma...@apache.org>: >>>> Author: markt >>>> Date: Mon Nov 17 08:06:24 2014 >>>> New Revision: 1640089 >>>> >>>> URL: http://svn.apache.org/r1640089 >>>> Log: >>>> Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=57216 >>>> Catch "/" as an invalid setting for context path and improve handling of >>>> null values. >>>> >>> >>> A number of test are failing at Gump. >>> (They are failing at Buildbot as well, but none e-mails were received). >>> >>> E.g. >>> Test org.apache.catalina.startup.TestListener FAILED >>> >>> and I see >>> [junit] 18-Nov-2014 01:21:53.447 WARNING [main] >>> org.apache.catalina.core.StandardContext.setPath A context path must >>> either be an empty string or start with a '/'. The path [] does not >>> meet these criteria and has been changed to [null] >> >> Fixed by r1640276 > > Thanks. > >> Remaining issues: >> a) I think that context paths that end with "/" shall be prohibited as well. > > Agreed.
Fixed. >> b) Run the whole testsuite and fix tests to do not use "/" as context path. > > Also agreed. Fixed. (In Tomcat 9, 8 and 7). Best regards, Konstantin Kolinko --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
