[jira] Updated: (WICKET-294) WebRequestCodingStrategy: path mounting and matching

2007-06-10 Thread Jean-Baptiste Quenot (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-294?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jean-Baptiste Quenot updated WICKET-294:


Fix Version/s: (was: trunk)
   1.3.0-beta1

> WebRequestCodingStrategy: path mounting and matching
> 
>
> Key: WICKET-294
> URL: https://issues.apache.org/jira/browse/WICKET-294
> Project: Wicket
>  Issue Type: Bug
>  Components: wicket
>Affects Versions: 2.0 branch (discontinued), trunk
>Reporter: Stephane Bailliez
>Assignee: Jean-Baptiste Quenot
> Fix For: 1.3.0-beta1
>
> Attachments: 
> 20070222-wicket-1.x-WebRequestCodingStrategy-path-matching.txt
>
>
> Assuming a mount path to "/p", it will match /pxyz
> Assuming this is the desired behavior of matching (warning), then to avoid 
> this match it should be declared "/p/" but it will create urls such as 
> '/app/p//SomePage'. which is wrong.
> In the servlet specs,  the mapping syntax '/p' is an exact match, this is not 
> what you want in your case since you're doing path mapping, so the syntax if 
> you want to stick close to the servlet specs should be '/p/*' or if you wan 
> to get close to mod_proxy syntax it would be '/p/'
> Note that the examples are also using this wrong mapping declaration. In the 
> example below: both should throw a 404:
> http://www.wicket-library.com/wicket-examples/niceurl/my/mounted/packageXXX
> http://www.wicket-library.com/wicket-examples/niceurl/my/mounted/Xpackage

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (WICKET-294) WebRequestCodingStrategy: path mounting and matching

2007-02-22 Thread Jean-Baptiste Quenot (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-294?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jean-Baptiste Quenot updated WICKET-294:


Attachment: 20070222-wicket-1.x-WebRequestCodingStrategy-path-matching.txt

What about the attached patch?  Can you please test and report?  I created a 
unit test for this.  However this does not solve the WicketRuntimeException 
when the class does not exist.

> WebRequestCodingStrategy: path mounting and matching
> 
>
> Key: WICKET-294
> URL: https://issues.apache.org/jira/browse/WICKET-294
> Project: Wicket
>  Issue Type: Bug
>  Components: wicket
>Affects Versions: 1.3, 2.0
>Reporter: Stephane Bailliez
> Assigned To: Jean-Baptiste Quenot
> Attachments: 
> 20070222-wicket-1.x-WebRequestCodingStrategy-path-matching.txt
>
>
> Assuming a mount path to "/p", it will match /pxyz
> Assuming this is the desired behavior of matching (warning), then to avoid 
> this match it should be declared "/p/" but it will create urls such as 
> '/app/p//SomePage'. which is wrong.
> In the servlet specs,  the mapping syntax '/p' is an exact match, this is not 
> what you want in your case since you're doing path mapping, so the syntax if 
> you want to stick close to the servlet specs should be '/p/*' or if you wan 
> to get close to mod_proxy syntax it would be '/p/'
> Note that the examples are also using this wrong mapping declaration. In the 
> example below: both should throw a 404:
> http://www.wicket-library.com/wicket-examples/niceurl/my/mounted/packageXXX
> http://www.wicket-library.com/wicket-examples/niceurl/my/mounted/Xpackage

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (WICKET-294) WebRequestCodingStrategy: path mounting and matching

2007-02-18 Thread Eelco Hillenius (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-294?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Eelco Hillenius updated WICKET-294:
---

Affects Version/s: 2.0

> WebRequestCodingStrategy: path mounting and matching
> 
>
> Key: WICKET-294
> URL: https://issues.apache.org/jira/browse/WICKET-294
> Project: Wicket
>  Issue Type: Bug
>  Components: wicket
>Affects Versions: 1.3, 2.0
>Reporter: Stephane Bailliez
>
> Assuming a mount path to "/p", it will match /pxyz
> Assuming this is the desired behavior of matching (warning), then to avoid 
> this match it should be declared "/p/" but it will create urls such as 
> '/app/p//SomePage'. which is wrong.
> In the servlet specs,  the mapping syntax '/p' is an exact match, this is not 
> what you want in your case since you're doing path mapping, so the syntax if 
> you want to stick close to the servlet specs should be '/p/*' or if you wan 
> to get close to mod_proxy syntax it would be '/p/'
> Note that the examples are also using this wrong mapping declaration. In the 
> example below: both should throw a 404:
> http://www.wicket-library.com/wicket-examples/niceurl/my/mounted/packageXXX
> http://www.wicket-library.com/wicket-examples/niceurl/my/mounted/Xpackage

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.