[servicecomb-java-chassis] branch master updated: [SCB-1230] avoid throw exception in httpClientResponse.exceptionHandler

[li...@apache.org]

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 6878d34  [SCB-1230] avoid throw exception in httpClientResponse.exceptionHandler
6878d34 is described below

commit 6878d343cb9e7b5ce2c975cf0afd26b25638f53e
Author: wujimin <wu...@huawei.com>
AuthorDate: Sat Mar 30 21:22:30 2019 +0800

    [SCB-1230] avoid throw exception in httpClientResponse.exceptionHandler
---
 .../transport/rest/client/http/RestClientInvocation.java           | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/RestClientInvocation.java b/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/RestClientInvocation.java
index 970e069..12d433d 100644
--- a/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/RestClientInvocation.java
+++ b/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/RestClientInvocation.java
@@ -249,7 +249,12 @@ public class RestClientInvocation {
     }
 
     stageTrace.finishClientFiltersResponse();
-    asyncResp.fail(invocation.getInvocationType(), e);
+
+    try {
+      asyncResp.fail(invocation.getInvocationType(), e);
+    } catch (Throwable e1) {
+      LOGGER.error(invocation.getMarker(), "failed to invoke asyncResp.fail.", e1);
+    }
   }
 
   protected void setCseContext() {