Yingyi Bu has submitted this change and it was merged.

Change subject: Avoid escaping HTML entities in error messages
......................................................................


Avoid escaping HTML entities in error messages

Change-Id: I385cad8f2fa28a9ae368e0e59b5257fa6325ac42
Reviewed-on: https://asterix-gerrit.ics.uci.edu/1568
Sonar-Qube: Jenkins <[email protected]>
Tested-by: Jenkins <[email protected]>
Integration-Tests: Jenkins <[email protected]>
Reviewed-by: Yingyi Bu <[email protected]>
---
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Yingyi Bu: Looks good to me, approved
  Jenkins: Verified; No violations found; Verified

Objections:
  Jenkins: Violations found



diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
index 3afb653..b730989 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
@@ -115,8 +115,8 @@
     public static void webUIErrorHandler(PrintWriter out, Exception e) {
         String errorTemplate = readTemplateFile("/webui/errortemplate.html", 
"%s\n%s\n%s");
 
-        String errorOutput = String.format(errorTemplate, 
escapeHTML(extractErrorMessage(e)),
-                escapeHTML(extractErrorSummary(e)), 
escapeHTML(extractFullStackTrace(e)));
+        String errorOutput =
+                String.format(errorTemplate, extractErrorMessage(e), 
extractErrorSummary(e), extractFullStackTrace(e));
         out.println(errorOutput);
     }
 

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1568
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I385cad8f2fa28a9ae368e0e59b5257fa6325ac42
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Till Westmann <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Xikui Wang <[email protected]>
Gerrit-Reviewer: Yingyi Bu <[email protected]>

Reply via email to