Skip to main content

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

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>
Expert advice, services and support from from the Jetty & CometD experts


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



Back to the top