Viktor Chmel created HTTPCORE-516:
-------------------------------------

             Summary: Reason phrase in HttpStatus constants are not needed due 
to changes in RFC 7230
                 Key: HTTPCORE-516
                 URL: https://issues.apache.org/jira/browse/HTTPCORE-516
             Project: HttpComponents HttpCore
          Issue Type: Task
          Components: HttpCore
            Reporter: Viktor Chmel


*Introduction*

After updating one (Serivce-A) of our Web services to Tomcat 8.5 we realised 
that HTTP Status header changed according to RFC 7230 

_RFC 7230 states that clients should ignore reason phrases in HTTP/1.1 response 
messages. Since the reason phrase is optional, Tomcat no longer sends it. As a 
result the system property org.apache.coyote.USE_CUSTOM_STATUS_MSG_IN_HEADER is 
no longer used and has been removed._ "

Some temporary workaround exists (set [deprecated option sendReasonPhrase to 
true|https://tomcat.apache.org/migration-85.html#HTTP_connector_changes]), but 
it will be impossible in tomcat 9.

*Problem*

Ok, what about HttpCore?

Another our service (Service-B) use [HttpStatus 
|https://hc.apache.org/httpcomponents-core-ga/httpcore/apidocs/org/apache/http/HttpStatus.html]lib
 and its constants like SC_OK to assert response from Service-A. And after 
updating to tomcat 8.5 it became to fail assertion every time. Why? Now SC_OK = 
"200 OK", but according to RFC 7230 must be just "200". The same issue with 
another status constans.

I`m not expert in you library, but smth must be done to support new standard 
from one hand and to support legacy http servers from another.

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to