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());