On 27/04/18 18:19, Christopher Schultz wrote: > Mark, > > On 4/27/18 1:15 PM, ma...@apache.org wrote: >> Author: markt Date: Fri Apr 27 17:15:00 2018 New Revision: 1830368 > >> URL: http://svn.apache.org/viewvc?rev=1830368&view=rev Log: >> Collapse multiple leading '/' characters to a single '/' in the >> return value of HttpServletRequest#getContextPath() to avoid issues >> if the value is used with HttpServletResponse#sendRedirect(). This >> behaviour is enabled by default and configurable via the new >> Context attribute allowMultipleLeadingForwardSlashInPath. > > Just confirming: the default behavior here matches that of the > existing behavior before the patch, yes? So this does not break > backward-compatibility ... it merely adds the option for disabling > that behavior which wasn't possible, before.
No. Tomcat now collapses leading slashes in the return value for HttpServletRequest#getContextPath() This is a change to the previous behaviour. It should be safe since Tomcat treats "///foo//bar" as "/foo/bar" It only broke some tests because they were looking for exact matches in the return values. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
