users@servlet-spec.java.net

[servlet-spec users] Re: [jsr369-experts] Re: Heads up: your action requested Re: Re: [73-getMapping] PROPOSAL

From: Mark Thomas <markt_at_apache.org>
Date: Mon, 27 Feb 2017 15:27:53 +0000

On 21/02/17 15:19, arjan tijms wrote:
> Hi,
>
> On Tue, Feb 21, 2017 at 3:46 PM, Edward Burns <edward.burns_at_oracle.com
> <mailto:edward.burns_at_oracle.com>> wrote:
>
> The above is incorrect according to the desires of Arjan and Mark. I
> have revised the table according to my understanding of the requested
> change. ACTION: Please check this!
>
> URI Path (in quotes) matchValue pattern mappingMatch
> "" "" / CONTEXT_ROOT
> "/MyServlet" MyServlet /MyServlet EXACT
> "/foo.extension" foo /*.extension EXTENSION
> "/path/foo" foo /path/* PATH
>
> If there are any other illustrative rows anyone would like to see in
> this table, please add them in reply to this email.
>
>
> This looks much better already, thanks!
>
> There's a small difference with Tomcat I believe as the matchValues have
> no leading slash here, but this may be better actually.

I'm fine with the match values without the leading '/'

> One thing is that for the EXTENSION mappinMatch the pattern shown is
> /*.extension, which should likely be *.extension.

Agreed.

> On other thing is that for CONTEXT_ROOT the pattern is shown as "/", but
> isn't that the pattern for DEFAULT?

It is. I'd expect the pattern to be ""

> At any length, an entry for the DEFAULT mapping would be good to add
> here as well.

Following the table above, I believe that should look like:

"/index.html" "" "/" DEFAULT


Mark