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 2023/09/21 11:43:47 UTC
[servicecomb-java-chassis] 01/01: [#3893]fix client decode error message
This is an automated email from the ASF dual-hosted git repository.
liubao pushed a commit to branch url_pattern
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git
commit 66bb27a429ced0afd96f2870f77b5428f7b33cec
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(),