plus sign in clean URL parameters not decoded since 1.5.1
---------------------------------------------------------

                 Key: STS-743
                 URL: http://www.stripesframework.org/jira/browse/STS-743
             Project: Stripes
          Issue Type: Bug
          Components: ActionBean Dispatching
    Affects Versions: Release 1.5.3, Release 1.5.2, Release 1.5.1
            Reporter: Ward van Wanrooij


Version 1.5.1 includes a change in the way parameters for a clean URL are 
parsed. Due to this change, the character + is no longer decoded to space as it 
should be (UrlBuilder generates these URLs).

Example for an actionbean bound to /action/hello/{name}:
version: URI => value of parameter name
v 1.5: /action/hello/John+Smith => John Smith
v 1.5: /action/hello/John%20Smith => John Smith
v 1.5.1+: /action/hello/John+Smith => John+Smith
v 1.5.1+: /action/hello/John%20Smith => John Smith

The incompatibility was introduced in UrlBindingFactory, by changing from 
trimContextPath to HttpUtil.getRequestedPath:

    public UrlBinding getBindingPrototype(HttpServletRequest request) {
//        return getBindingPrototype(trimContextPath(request));
        return getBindingPrototype(HttpUtil.getRequestedPath(request));
    }

    public UrlBinding getBinding(HttpServletRequest request) {
//       return getBinding(trimContextPath(request));
        return getBinding(HttpUtil.getRequestedPath(request));
    }


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://www.stripesframework.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to