[jira] [Commented] (OFBIZ-10569) Wrong handling of response type "cross-redirect"

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-10569) Wrong handling of response type "cross-redirect"

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-10569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16642956#comment-16642956 ]

Dennis Balkir commented on OFBIZ-10569:
---------------------------------------

Hi Jacques,

yes please create another issue for this.

This will be a great feature to have and we can use the patch I made with slight modifications as a starting point, since the modified code already provides the feature.
We just have to change it up a bit, to make it a new type

> Wrong handling of response type "cross-redirect"
> ------------------------------------------------
>
>                 Key: OFBIZ-10569
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10569
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>            Reporter: Dennis Balkir
>            Assignee: Jacques Le Roux
>            Priority: Minor
>             Fix For: 17.12.01, 16.11.06
>
>         Attachments: OFBIZ-10569_RequestHandler.patch
>
>
> When using the response type "cross-redirect", OFBiz is not acting as expected.
> The description of the types "url" and "cross-redirect" is stated as follows in the file {{site-conf.xsd}}:
> {code:xml}
> <xs:enumeration value="url">
>   <xs:annotation>
>     <xs:documentation>
>       Any URL, relative or absolute. Redirected parameters are not used, you can put them in the url.
>     </xs:documentation>
>   </xs:annotation>
> </xs:enumeration>
> <xs:enumeration value="cross-redirect">
>   <xs:annotation>
>     <xs:documentation>
>       Works like URL but you can also pass redirected parameters.
>     </xs:documentation>
>   </xs:annotation>
> </xs:enumeration>
> {code}
> The way, this is implemented in {{RequestHandler.java}} works slightly different:
> For the type "url", this is true, but for "cross-redirect", before putting the params in the link, it is checked, if the link begins with an "/".
> If not, it is added as the first char.
> This results in a link given like this: {{https://test.com}}
> will turn into this: {{/https://test.com}}, which will then be treated as an OFBiz internal link, like {{/catalog}}, and instead of be called externally, OFBiz tries to call it as an OFBiz-URL.
> This behavior is not correct and should therefore be corrected.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)