You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicecomb.apache.org by li...@apache.org on 2021/09/18 08:04:30 UTC

[servicecomb-java-chassis] branch master updated: [SCB-2339]HttpServletResponseEx implements setStatus (#2592)

This is an automated email from the ASF dual-hosted git repository.

liubao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git


The following commit(s) were added to refs/heads/master by this push:
     new ad0dc90  [SCB-2339]HttpServletResponseEx implements setStatus (#2592)
ad0dc90 is described below

commit ad0dc90b8c1219de5189c642e0191909734bf3c1
Author: liubao68 <bi...@qq.com>
AuthorDate: Sat Sep 18 16:04:27 2021 +0800

    [SCB-2339]HttpServletResponseEx implements setStatus (#2592)
---
 .../foundation/vertx/http/StandardHttpServletResponseEx.java       | 7 +++++++
 .../vertx/http/VertxServerResponseToHttpServletResponse.java       | 5 +++++
 2 files changed, 12 insertions(+)

diff --git a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/StandardHttpServletResponseEx.java b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/StandardHttpServletResponseEx.java
index 8a00cb8..ba1ee63 100644
--- a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/StandardHttpServletResponseEx.java
+++ b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/StandardHttpServletResponseEx.java
@@ -26,6 +26,7 @@ import java.util.concurrent.CompletableFuture;
 import javax.servlet.http.HttpServletResponse;
 import javax.servlet.http.HttpServletResponseWrapper;
 import javax.servlet.http.Part;
+import javax.ws.rs.core.Response.Status;
 import javax.ws.rs.core.Response.StatusType;
 
 import org.apache.servicecomb.foundation.common.http.HttpStatus;
@@ -74,6 +75,12 @@ public class StandardHttpServletResponseEx extends HttpServletResponseWrapper im
   }
 
   @Override
+  public void setStatus(int sc) {
+    super.setStatus(sc);
+    statusType = Status.fromStatusCode(sc);
+  }
+
+  @Override
   public int getStatus() {
     return statusType.getStatusCode();
   }
diff --git a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java
index 9fdb51c..23e8bb7 100644
--- a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java
+++ b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/http/VertxServerResponseToHttpServletResponse.java
@@ -59,6 +59,11 @@ public class VertxServerResponseToHttpServletResponse extends AbstractHttpServle
   }
 
   @Override
+  public void setStatus(int sc) {
+    serverResponse.setStatusCode(sc);
+  }
+  
+  @Override
   public StatusType getStatusType() {
     if (statusType == null) {
       statusType = new HttpStatus(serverResponse.getStatusCode(), serverResponse.getStatusMessage());