[ http://jira.magnolia.info/browse/MAGNOLIA-2115?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gregory Joseph updated MAGNOLIA-2115: ------------------------------------- Fix Version/s: 3.6.1 (was: 3.6) Assignee: Gregory Joseph (was: Philipp Bracher) Argh - sorry I didn't see there was a patch with this issue earlier - will have to go through it after the final, hopefully will include in 3.6.1 - need to make sure nothing breaks. > The ServletDispatchingFilter doesn't support the servlet spec for mapping of > servlets > ------------------------------------------------------------------------------------- > > Key: MAGNOLIA-2115 > URL: http://jira.magnolia.info/browse/MAGNOLIA-2115 > Project: Magnolia > Issue Type: Improvement > Components: core > Affects Versions: 3.5.4 > Reporter: Tom Jensen > Assignee: Gregory Joseph > Priority: Minor > Fix For: 3.6.1 > > Attachments: servletFilterFix.patch > > > Proposed implementation: > 1) A basic mapping must adhere to the servlet spec exactly (i.e., /test* > would require the url passed from the browser to literally be '/test*') > servletPath and pathInfo are split according to the spec. > 2) A mapping prepended with 'regex:' is treated as a regular expression but > the servletPath and pathInfo are more complex: > a) The servletPath will always include everything that is within the > mapping url with the following exception > b) If the regex ends with a '/' it is included, along with any other > characters to the end of the request url as part of the pathInfo > c) No groupings (parentheses) are allowed unless escaped to be treated as > literals > See attached patch for implementation of this with updated tests. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.magnolia.info/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira ---------------------------------------------------------------- for list details see http://documentation.magnolia.info/ ----------------------------------------------------------------