[GitHub] [cxf] reta commented on a change in pull request #889: CXF-8635: Fix org.apache.cxf.jaxrs.client.logging.RESTLoggingTest.testSlf4

[GitBox <gi...@apache.org>]

reta commented on a change in pull request #889:
URL: https://github.com/apache/cxf/pull/889#discussion_r776109741



##########
File path: rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/AbstractClient.java
##########
@@ -440,8 +440,7 @@ protected ResponseBuilder setResponseBuilder(Message outMessage, Exchange exchan
             return currentResponseBuilder;
         }
         
-        String reasonPhrase = (String)MessageUtils.getContextualProperty(
-                   responseMessage, HTTPConduit.HTTP_RESPONSE_MESSAGE, null);
+        final String reasonPhrase = (String)responseMessage.get(HTTPConduit.HTTP_RESPONSE_MESSAGE);

Review comment:
       @ffang if you could take a look please, it seems like we should fetch the `HTTPConduit.HTTP_RESPONSE_MESSAGE` from the message itself, not the contextual property, thank you.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org