[jira] Commented: (WICKET-2777) HybridUrlCodingStrategy incorrectly decodes url parameters on redirect
[ https://issues.apache.org/jira/browse/WICKET-2777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12903798#action_12903798 ] Peter Ertl commented on WICKET-2777: Please: - attach a quickstart - check this behavior against the current version of wicket (it might be fixed already) > HybridUrlCodingStrategy incorrectly decodes url parameters on redirect > -- > > Key: WICKET-2777 > URL: https://issues.apache.org/jira/browse/WICKET-2777 > Project: Wicket > Issue Type: Bug >Affects Versions: 1.4.6, 1.4.7 >Reporter: Pointbreak > > When using HybridUrlCodingStrategy and encoded parameters the following > happens: > - In some cases HybridUrlCodingStrategy does a page-redirect to get an > explicit version nr (i.e. http://localhost/page gets redirected to > http://localhost/page.5). > - However the redirect URL is not the same before the version nr as the > original URL, part of the URL path gets urldecoded, resulting in > page-parameters that are different from the ones in the original link. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (WICKET-2777) HybridUrlCodingStrategy incorrectly decodes url parameters on redirect
[ https://issues.apache.org/jira/browse/WICKET-2777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12879573#action_12879573 ] Russell Morrisey commented on WICKET-2777: -- In my specific use case, what I thought was an example of this issue turned out to be a bug in my own code. Apologies! I've removed my previous comments. > HybridUrlCodingStrategy incorrectly decodes url parameters on redirect > -- > > Key: WICKET-2777 > URL: https://issues.apache.org/jira/browse/WICKET-2777 > Project: Wicket > Issue Type: Bug >Affects Versions: 1.4.6, 1.4.7 >Reporter: Pointbreak > > When using HybridUrlCodingStrategy and encoded parameters the following > happens: > - In some cases HybridUrlCodingStrategy does a page-redirect to get an > explicit version nr (i.e. http://localhost/page gets redirected to > http://localhost/page.5). > - However the redirect URL is not the same before the version nr as the > original URL, part of the URL path gets urldecoded, resulting in > page-parameters that are different from the ones in the original link. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (WICKET-2777) HybridUrlCodingStrategy incorrectly decodes url parameters on redirect
[ https://issues.apache.org/jira/browse/WICKET-2777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12879561#action_12879561 ] Russell Morrisey commented on WICKET-2777: -- I am still seeing this issue in 1.4.9. The following code in BookmarkablePageRequestTarget is suspect: // Add bookmarkable params in for WICKET-400. final Map requestMap = requestCycle.getRequest().getParameterMap(); requestMap.putAll(pageParameters.toRequestParameters()); return pageFactory.newPage(pageClass, pageParameters); I am thinking that it should be: return pageFactory.newPage(pageClass, requestMap); //use the new map we just constructed > HybridUrlCodingStrategy incorrectly decodes url parameters on redirect > -- > > Key: WICKET-2777 > URL: https://issues.apache.org/jira/browse/WICKET-2777 > Project: Wicket > Issue Type: Bug >Affects Versions: 1.4.6, 1.4.7 >Reporter: Pointbreak > > When using HybridUrlCodingStrategy and encoded parameters the following > happens: > - In some cases HybridUrlCodingStrategy does a page-redirect to get an > explicit version nr (i.e. http://localhost/page gets redirected to > http://localhost/page.5). > - However the redirect URL is not the same before the version nr as the > original URL, part of the URL path gets urldecoded, resulting in > page-parameters that are different from the ones in the original link. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (WICKET-2777) HybridUrlCodingStrategy incorrectly decodes url parameters on redirect
[ https://issues.apache.org/jira/browse/WICKET-2777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12879557#action_12879557 ] Russell Morrisey commented on WICKET-2777: -- I am experiencing this issue in 1.4.5; it looks like it may be a duplicate of WICKET-2717; however, the "affects versions" for this ticket says it affects 1.4.7, and 2717 was fixed in 1.4.7 according to the other ticket. > HybridUrlCodingStrategy incorrectly decodes url parameters on redirect > -- > > Key: WICKET-2777 > URL: https://issues.apache.org/jira/browse/WICKET-2777 > Project: Wicket > Issue Type: Bug >Affects Versions: 1.4.6, 1.4.7 >Reporter: Pointbreak > > When using HybridUrlCodingStrategy and encoded parameters the following > happens: > - In some cases HybridUrlCodingStrategy does a page-redirect to get an > explicit version nr (i.e. http://localhost/page gets redirected to > http://localhost/page.5). > - However the redirect URL is not the same before the version nr as the > original URL, part of the URL path gets urldecoded, resulting in > page-parameters that are different from the ones in the original link. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Commented: (WICKET-2777) HybridUrlCodingStrategy incorrectly decodes url parameters on redirect
[ https://issues.apache.org/jira/browse/WICKET-2777?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12844122#action_12844122 ] Martin Grigorov commented on WICKET-2777: - Attach a quickstart application please. I'll try to provide a patch. > HybridUrlCodingStrategy incorrectly decodes url parameters on redirect > -- > > Key: WICKET-2777 > URL: https://issues.apache.org/jira/browse/WICKET-2777 > Project: Wicket > Issue Type: Bug >Affects Versions: 1.4.6, 1.4.7 >Reporter: Pointbreak > > When using HybridUrlCodingStrategy and encoded parameters the following > happens: > - In some cases HybridUrlCodingStrategy does a page-redirect to get an > explicit version nr (i.e. http://localhost/page gets redirected to > http://localhost/page.5). > - However the redirect URL is not the same before the version nr as the > original URL, part of the URL path gets urldecoded, resulting in > page-parameters that are different from the ones in the original link. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.