Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jetty-users] Bug in PathMap

Thanks.

Will that be in 9.0.6?


On Sun, Sep 8, 2013 at 9:51 PM, Jan Bartel <janb@xxxxxxxxxxx> wrote:
Nils,

I fixed https://bugs.eclipse.org/bugs/show_bug.cgi?id=416787. Yes, you
can have "" as the url mapping for a servlet or filter from servlet
spec 3.0 onwards.

cheers
Jan

On 8 September 2013 11:58, Nils Kilden-Pedersen
<nils@xxxxxxxxxxxxxxxxxxx> wrote:
> On Sat, Sep 7, 2013 at 8:54 PM, Nils Kilden-Pedersen
> <nils@xxxxxxxxxxxxxxxxxxx> wrote:
>>
>> On Sat, Sep 7, 2013 at 6:49 PM, Joakim Erdfelt <joakim@xxxxxxxxxxx> wrote:
>>>
>>> That is a bug, and should be fixed.
>>>
>>> However, Servlet Spec 3.1 pretty much banned "" as a path spec (stating
>>> it as a mistake of past specs) and now enforces the use of "/".
>>
>>
>> That's interesting, considering the "" context root addition
>> (clarification?) wasn't added until the 3.0 spec, where "" maps exactly to
>> the context root and "/" maps to the default servlet. Has this been changed?
>> In other words, how is that distinction made in 3.1?
>
> I did the obvious thing and check the 3.1 spec, which states the exact same
> thing as 3.0. From section 12.2:
>
> The empty string (“”) is a special URL pattern that exactly maps to the
> application's context root, i.e., requests of the form http://host:port//.
> In this case the path info is ’/’ and the servlet path and context path is
> empty string (““).
>
> I don't see any indication that "" is banned as a path spec?
>>
>>
>>>
>>> If you haven't updated your code yet to use "/", you should stick with
>>> Jetty 7/8, as Jetty 9 is going to be Servlet Spec 3.1 moving forward.
>>
>>
>> I'm not live yet, building on 9. So I'm really waiting for 9.1 to be
>> released.
>>
>>>
>>>
>>> --
>>> Joakim Erdfelt <joakim@xxxxxxxxxxx>
>>> webtide.com - intalio.com/jetty
>>> Expert advice, services and support from from the Jetty & CometD experts
>>> eclipse.org/jetty - cometd.org
>>>
>>>
>>> On Sat, Sep 7, 2013 at 12:38 PM, Nils Kilden-Pedersen
>>> <nils@xxxxxxxxxxxxxxxxxxx> wrote:
>>>>
>>>> I think.
>>>>
>>>> Got hit with a StringIndexOutOfBounds something.
>>>>
>>>> Tracked it to line 404 (yeah, really) in PathMap:
>>>>
>>>> char c = pathSpec.charAt(0);
>>>>
>>>> Since Servlet spec 3.0, empty strings "" matches on the context root. I
>>>> had a filter defined with urlPatterns={""}.
>>>>
>>>> I can‘t quite figure out the validity of matching filters to context
>>>> root, since I’ve seen similar reports from WebSphere and Glassfish, but
>>>> regardless, the error message should be better then.
>>>>
>>>> Nils
>>>>
>>>>
>>>> _______________________________________________
>>>> jetty-users mailing list
>>>> jetty-users@xxxxxxxxxxx
>>>> https://dev.eclipse.org/mailman/listinfo/jetty-users
>>>>
>>>
>>>
>>> _______________________________________________
>>> jetty-users mailing list
>>> jetty-users@xxxxxxxxxxx
>>> https://dev.eclipse.org/mailman/listinfo/jetty-users
>>>
>>
>
> _______________________________________________
> jetty-users mailing list
> jetty-users@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/jetty-users
>



--
Jan Bartel <janb@xxxxxxxxxxx>
www.webtide.com
'Expert Jetty/CometD developer,production,operations advice'


Back to the top