Till Westmann has submitted this change and it was merged. Change subject: Set response status for web interface servlets ......................................................................
Set response status for web interface servlets Change-Id: Id4b5e153a8c589b666a44ae17bb05c69cfc3c9d9 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1528 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Reviewed-by: Till Westmann <[email protected]> --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/ApiServlet.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Till Westmann: 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/api/http/server/ApiServlet.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/ApiServlet.java index 234d134..7a33c0e 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/ApiServlet.java +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/ApiServlet.java @@ -113,6 +113,7 @@ String printJob = request.getParameter("print-job"); String executeQuery = request.getParameter("execute-query"); try { + response.setStatus(HttpResponseStatus.OK); HttpUtil.setContentType(response, ContentType.TEXT_HTML, Encoding.UTF8); } catch (IOException e) { LOGGER.log(Level.WARNING, "Failure setting content type", e); @@ -162,7 +163,7 @@ protected void get(IServletRequest request, IServletResponse response) { String resourcePath = null; String requestURI = request.getHttpRequest().uri(); - + response.setStatus(HttpResponseStatus.OK); if ("/".equals(requestURI)) { try { HttpUtil.setContentType(response, HttpUtil.ContentType.TEXT_HTML, HttpUtil.Encoding.UTF8); -- To view, visit https://asterix-gerrit.ics.uci.edu/1528 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4b5e153a8c589b666a44ae17bb05c69cfc3c9d9 Gerrit-PatchSet: 4 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]>
