I think the existence of the the query parameters in the getPath() was a mistake. a bug.
Also think the API is being aligned to match behavior in the Servlet Spec and the Java WebSocket API (both which exclude query parameters from paths and request URIs, and instead have a getQuery() method to specifically get the query string) HttpClient version: HttpRequest.getQuery() - http://download.eclipse.org/jetty/stable-9/apidocs/org/eclipse/jetty/client/HttpRequest.html#getQuery() javax.servlet.http.HttpServletRequest version: .getQueryString() - http://docs.oracle.com/javaee/6/api/javax/servlet/http/HttpServletRequest.html#getQueryString() javax.websocket.server.HandshakeRequest version: .getQueryString() - http://javaee-spec.java.net/nonav/javadocs/javax/websocket/server/HandshakeRequest.html#getQueryString() org.eclipse.jetty.websocket.api.UpgradeRequest version: .getQueryString() - http://download.eclipse.org/jetty/stable-9/apidocs/org/eclipse/jetty/websocket/api/UpgradeRequest.html#getQueryString() -- Joakim Erdfelt <[email protected]> webtide.com <http://www.webtide.com/> Developer advice, services and support from the Jetty & CometD experts eclipse.org/jetty - cometd.org On Fri, May 10, 2013 at 1:07 PM, David Kellum <[email protected]> wrote: > It might be worth noting that 9.0.3.v20130506 appears to introduce an > interface change, which is possibly also a desirably fix, in that > org.eclipse.jetty.client.api.**Request.getPath() no longer includes any > query string parameters that might be present. I'm guessing this had > something to do with: > > jetty-9.0.3.v20130506 06 May 2013 >> + 406015 Query parameters and POST queries. Fixed proxy case where the >> path is >> rewritten to be absolute. >> > > But this particular change is broader than only proxy requests. I was > relying on query parameters being in getPath() through 9.0.2. A workaround > that makes user code work for all 9.0.x versions, 9.0.3 included, is to use > the similar accessors available via Request.getURI() -> URI. > > Thanks, > David > > ______________________________**_________________ > jetty-users mailing list > [email protected] > https://dev.eclipse.org/**mailman/listinfo/jetty-users<https://dev.eclipse.org/mailman/listinfo/jetty-users> >
_______________________________________________ jetty-users mailing list [email protected] https://dev.eclipse.org/mailman/listinfo/jetty-users
