| commit | 223ec90a4580fbbea01d410e1056e83951a6375e | [log] [tgz] |
|---|---|---|
| author | Michael Blow <mblow@apache.org> | Fri Jun 09 02:35:38 2017 -0400 |
| committer | Michael Blow <mblow@apache.org> | Fri Jun 09 19:19:49 2017 -0700 |
| tree | 554d57a2f71ac5de5fa0cbd33d8916287b4736a8 | |
| parent | cb0c265fcc3e8c0a2741e35be36dee5998677a1c [diff] |
Strip query string from servlet localPath Path should not contain query string; remove it when extracting localPath from request uri. Change-Id: I0b4942fb1249807fcee9d3047568dc1749798d85 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1819 Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu> Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu> Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu> BAD: Jenkins <jenkins@fulliautomatix.ics.uci.edu> Reviewed-by: Till Westmann <tillw@apache.org>
diff --git a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/AbstractServlet.java b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/AbstractServlet.java index d029bbc..1030686 100644 --- a/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/AbstractServlet.java +++ b/hyracks-fullstack/hyracks/hyracks-http/src/main/java/org/apache/hyracks/http/server/AbstractServlet.java
@@ -139,7 +139,8 @@ public String localPath(IServletRequest request) { final String uri = request.getHttpRequest().uri(); - return uri.substring(trim(uri)); + int queryStart = uri.indexOf("?"); + return queryStart == -1 ? uri.substring(trim(uri)) : uri.substring(trim(uri), queryStart); } public String servletPath(IServletRequest request) {