Hi,
On Fri, Feb 17, 2017 at 9:34 PM, Edward Burns <edward.burns_at_oracle.com>
wrote:
> >>>>> On Fri, 17 Feb 2017 14:47:55 +0000, Mark Thomas <markt_at_apache.org>
> said:
> AT> Calling with http://localhost:8080/servlet4/bla.ext
>
> AT> Prints:
>
> AT> Mapping match:EXTENSION
> AT> Match value:/bla
>
> EB> In my implementation in GlassFish I have this returning /bla.ext.
>
> AT> Pattern:*.ext
>
> MT> My understanding was that the 'Match value' was meant to be what the
> '*'
> MT> was matched with.
>
> AT> Calling with http://localhost:8080/servlet4/path/bla
>
> AT> Prints:
>
> AT> Mapping match:PATH
> AT> Match value:/bla
>
> EB> GlassFish: /path/bla
>
> AT> Pattern:/path/*
>
> MT> Same as previous point.
>
I'd much prefer the Tomcat interpretation of the match value. With that the
unique match is far easier to distinguish as it's indeed what the '*' was
matched with.
Match value and Pattern can be relatively easy combined if needed. The
subtraction of the pattern from the match is a far less easy operation.
Kind regards,
Arjan Tijms
> MT> Arjan can comment better than I how significant this is. My default
> MT> position is to support whichever Arjan prefers.
>
> >>>>> On Fri, 17 Feb 2017 12:32:20 -0800, Edward Burns <
> edward.burns_at_oracle.com> said:
>
> EB> I would like to argue for the full string that matches the pattern.
>
> EB> ACTION: Please review the API at [1] and comment by close of business
> EB> GMT-0800 Tuesday 21 February.
>
> MT> On my TODO list.
>
> EB> Thank you. You might want to skip right to the table in
>
> EB> https://maven.java.net/service/local/repositories/
> snapshots/archive/javax/servlet/javax.servlet-api/4.0.
> 0-b03-SERVLET_SPEC-73-SNAPSHOT/javax.servlet-api-4.
> 0.0-b03-SERVLET_SPEC-73-20170216.222658-3-javadoc.jar/
> !/javax/servlet/http/ServletMapping.html
>
> EB> Sorry for the truncated lines in my last email. I can resend it if you
> EB> like.
>
> --
> | edward.burns_at_oracle.com | office: +1 407 458 0017
> | 12 business days until planned start of JSF 2.3 Final Approval Ballot
> | 2 business days until DevNexus 2017
> | 27 business days until JavaLand 2017
>