commit d21ee1980f6db7a0845e6f9732471909993a205c
author Edwin Kempin <ekempin@google.com> Fri Jan 17 15:22:30 2020 +0100
committer Edwin Kempin <ekempin@google.com> Fri Jan 17 15:23:50 2020 +0100
tree 384c0ad668f1a9a2e11b6ef00fa74fc74d55ed05
parent ed80360d7ec515c21db57d132b7388c7e3a31f1d

RestApiServlet: Do not use HttpServletResponse#getStatus()

The version of HttpServletResponse that is used at Google doesn't have
this method. The usage of this method was introduced by change
If6fd0b543.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I9035f2a7e874fe5341eae5a43a9d5e4cfe0dae08

java/com/google/gerrit/httpd/restapi/RestApiServlet.java

diff --git a/java/com/google/gerrit/httpd/restapi/RestApiServlet.java b/java/com/google/gerrit/httpd/restapi/RestApiServlet.java
index cca4f69..f30fefa 100644
--- a/java/com/google/gerrit/httpd/restapi/RestApiServlet.java
+++ b/java/com/google/gerrit/httpd/restapi/RestApiServlet.java
@@ -685,12 +685,11 @@
           responseBytes = 0;
           if (statusCode == SC_INTERNAL_SERVER_ERROR) {
             logger.atSevere().withCause(e).log(
-                "Error in %s %s, response already committed with status %d",
-                req.getMethod(), uriForLogging(req), res.getStatus());
+                "Error in %s %s, response already committed", req.getMethod(), uriForLogging(req));
           } else {
             logger.atWarning().log(
-                "Response for %s %s already committed with status %d, wanted to set status %d",
-                req.getMethod(), uriForLogging(req), res.getStatus(), statusCode);
+                "Response for %s %s already committed, wanted to set status %d",
+                req.getMethod(), uriForLogging(req), statusCode);
           }
         } else {
           res.reset();