diff options
author | Matthias Sohn <matthias.sohn@sap.com> | 2021-01-14 22:07:27 +0100 |
---|---|---|
committer | Matthias Sohn <matthias.sohn@gmail.com> | 2021-01-14 21:17:01 +0000 |
commit | 69b728ba958883dcfadf4512c088b2c27e1ecb8e (patch) | |
tree | b25b58f1ddc8614048f3e6a24078d15416db050d | |
parent | bd675456bb3369ccdf066d9de5c16a85da527234 (diff) |
Fix badly formatted error message shown in error dialog
Since [1] error messages which look like html are encoded as json to
workaround some limitations of old MSIE versions. This led to badly
formatted error messages shown in an error dialog.
Fix this by removing this json encoding.
[1] https://gerrit-review.googlesource.com/c/gerrit/+/39653
Bug: Issue 13936
Change-Id: I1bb7ddfc006d77b7a55727b5512dce6b67054aee
-rw-r--r-- | java/com/google/gerrit/httpd/restapi/RestApiServlet.java | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/java/com/google/gerrit/httpd/restapi/RestApiServlet.java b/java/com/google/gerrit/httpd/restapi/RestApiServlet.java index a7c0e300fd..fe3621ed19 100644 --- a/java/com/google/gerrit/httpd/restapi/RestApiServlet.java +++ b/java/com/google/gerrit/httpd/restapi/RestApiServlet.java @@ -47,7 +47,6 @@ import static javax.servlet.http.HttpServletResponse.SC_PRECONDITION_FAILED; import static javax.servlet.http.HttpServletResponse.SC_SERVICE_UNAVAILABLE; import com.google.common.annotations.VisibleForTesting; -import com.google.common.base.CharMatcher; import com.google.common.base.Joiner; import com.google.common.base.Splitter; import com.google.common.base.Strings; @@ -134,7 +133,6 @@ import com.google.gson.Gson; import com.google.gson.GsonBuilder; import com.google.gson.JsonElement; import com.google.gson.JsonParseException; -import com.google.gson.JsonPrimitive; import com.google.gson.stream.JsonReader; import com.google.gson.stream.JsonToken; import com.google.gson.stream.JsonWriter; @@ -1549,10 +1547,6 @@ public class RestApiServlet extends HttpServlet { static long replyText( @Nullable HttpServletRequest req, HttpServletResponse res, boolean allowTracing, String text) throws IOException { - if ((req == null || isRead(req)) && isMaybeHTML(text)) { - return replyJson( - req, res, allowTracing, ImmutableListMultimap.of("pp", "0"), new JsonPrimitive(text)); - } if (!text.endsWith("\n")) { text += "\n"; } @@ -1562,10 +1556,6 @@ public class RestApiServlet extends HttpServlet { return replyBinaryResult(req, res, BinaryResult.create(text).setContentType(PLAIN_TEXT)); } - private static boolean isMaybeHTML(String text) { - return CharMatcher.anyOf("<&").matchesAnyOf(text); - } - private static boolean acceptsGzip(HttpServletRequest req) { if (req != null) { String accepts = req.getHeader(HttpHeaders.ACCEPT_ENCODING); |