google / google/guice

invalid pathinfo computation in ServletDefinition

Open
#599 6 comments 0 reactions 0 assignees View on GitHub
Extension-Servlet imported
Dominant language
Java
Stars
12.7k
Forks
1.7k
Avg merge
11m
Merged PRs (30d)
2

Description

_From [mathieu.barcikowski](https://code.google.com/u/110838961409494838407/) on February 02, 2011 08:25:01_

When a path end up with ";" params (for example,
http://server/servlet/path;jsessionid=xxxxxx
the getPathInfo method in the doService method return "/path;jsessionid=xxxxxx" instead of "/path".

It's not consistent with the way jetty (for example) treat this particular case), jetty discard the ";jsessionid=xxxxxx" part when computing the pathinfo.

This difference leads to a problem when using the blazeds MessageBrokerServlet managed by servlet extension, because it using servletpath+pathinfo to retrieve the amf endpoint, and thus it don't retrieve the endpoint.

How To reproduce :
- configure a MessageBrokerServlet in a ServletModule (serving "/messagebroker/*" for example)
- configure an endpoint in the blazeds side (for example "http://{server.name}:{server.port}/{context.root}/messagebroker/amf/
- now start the server and try to access endpoint :
  * http://server/context/messagebroker/amf return a HTTP status 200
  * http://server/context/messagebroker/amf;jsessionid=112121 return a HTTP status 404

_Original issue: http://code.google.com/p/google-guice/issues/detail?id=599_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.