aboutsummaryrefslogtreecommitdiffstats
path: root/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/HttpRequestFactory.java
diff options
context:
space:
mode:
Diffstat (limited to 'jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/HttpRequestFactory.java')
-rw-r--r--jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/HttpRequestFactory.java50
1 files changed, 1 insertions, 49 deletions
diff --git a/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/HttpRequestFactory.java b/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/HttpRequestFactory.java
index 95f26e8bc1b..617e081bd24 100644
--- a/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/HttpRequestFactory.java
+++ b/jdisc_http_service/src/main/java/com/yahoo/jdisc/http/server/jetty/HttpRequestFactory.java
@@ -38,7 +38,7 @@ class HttpRequestFactory {
}
public static URI getUri(HttpServletRequest servletRequest) {
- String query = extraQuote(servletRequest.getQueryString());
+ String query = servletRequest.getQueryString();
try {
return URI.create(servletRequest.getRequestURL() + (query != null ? '?' + query : ""));
} catch (IllegalArgumentException e) {
@@ -59,54 +59,6 @@ class HttpRequestFactory {
}
}
- // TODO Remove this ugly, non-complete escaping in Vespa 7
- private static String extraQuote(String queryString) {
- // TODO: Use an URI builder
- if (queryString == null) return null;
-
- int toAndIncluding = -1;
- for (int i = 0; i < queryString.length(); ++i) {
- if (quote(queryString.charAt(i)) != null) {
- break;
- }
- toAndIncluding = i;
- }
-
- String washed;
- if (toAndIncluding != (queryString.length() - 1)) {
- StringBuilder w = new StringBuilder(queryString.substring(0, toAndIncluding + 1));
- for (int i = toAndIncluding + 1; i < queryString.length(); ++i) {
- String s = quote(queryString.charAt(i));
- if (s == null) {
- w.append(queryString.charAt(i));
- } else {
- w.append(s);
- }
- }
- washed = w.toString();
- } else {
- washed = queryString;
- }
- return washed;
- }
-
- private static String quote(char c) {
- switch(c) {
- case '\\':
- return "%5C";
- case '^':
- return "%5E";
- case '{':
- return "%7B";
- case '|':
- return "%7C";
- case '}':
- return "%7D";
- default:
- return null;
- }
- }
-
private static X509Certificate[] getCertChain(HttpServletRequest servletRequest) {
return (X509Certificate[]) servletRequest.getAttribute("javax.servlet.request.X509Certificate");
}