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