[servicecomb-java-chassis] 01/02: [#3893]fix client decode error message

[liubao]

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

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

commit b634c4a9b612c7adbe66510d8637c3de9e3199a8
Author: liubao <bi...@qq.com>
AuthorDate: Thu Sep 21 19:41:55 2023 +0800

    [#3893]fix client decode error message
---
 .../rest/client/http/DefaultHttpClientFilter.java  | 25 +++++++++-------------
 1 file changed, 10 insertions(+), 15 deletions(-)

diff --git 
a/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/DefaultHttpClientFilter.java
 
b/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/DefaultHttpClientFilter.java
index 9e9c7d7ce..91b98a48a 100644
--- 
a/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/DefaultHttpClientFilter.java
+++ 
b/transports/transport-rest/transport-rest-client/src/main/java/org/apache/servicecomb/transport/rest/client/http/DefaultHttpClientFilter.java
@@ -90,14 +90,13 @@ public class DefaultHttpClientFilter implements 
HttpClientFilter {
     if (produceProcessor == null) {
       // This happens outside the runtime such as Servlet filter response. 
Here we give a default json parser to it
       // and keep user data not get lose.
-      String msg =
-          String.format("method %s, path %s, statusCode %d, reasonPhrase %s, 
response content-type %s is not supported",
-              swaggerRestOperation.getHttpMethod(),
-              swaggerRestOperation.getAbsolutePath(),
-              responseEx.getStatus(),
-              responseEx.getStatusType().getReasonPhrase(),
-              responseEx.getHeader(HttpHeaders.CONTENT_TYPE));
-      LOGGER.warn(msg);
+      LOGGER.warn("Response content-type {} is not supported. Method {}, path 
{}, statusCode {}, reasonPhrase {}.",
+          responseEx.getHeader(HttpHeaders.CONTENT_TYPE),
+          swaggerRestOperation.getHttpMethod(),
+          swaggerRestOperation.getAbsolutePath(),
+          responseEx.getStatus(),
+          responseEx.getStatusType().getReasonPhrase()
+      );
       produceProcessor = 
ProduceProcessorManager.INSTANCE.findDefaultProcessor();
     }
 
@@ -110,14 +109,10 @@ public class DefaultHttpClientFilter implements 
HttpClientFilter {
       }
       return response;
     } catch (Exception e) {
-      LOGGER.error("failed to decode response body, exception is [{}]", 
e.getMessage());
+      LOGGER.error("failed to decode response body.", e);
       String msg =
-          String.format("method %s, path %s, statusCode %d, reasonPhrase %s, 
response content-type %s is not supported",
-              swaggerRestOperation.getHttpMethod(),
-              swaggerRestOperation.getAbsolutePath(),
-              responseEx.getStatus(),
-              responseEx.getStatusType().getReasonPhrase(),
-              responseEx.getHeader(HttpHeaders.CONTENT_TYPE));
+          String.format("Failed to decode response body. %s",
+              e.getMessage());
       if (HttpStatus.isSuccess(responseEx.getStatus())) {
         return Response.createConsumerFail(
             new InvocationException(400, 
responseEx.getStatusType().getReasonPhrase(),