zendframework / zend-diactoros

PSR-7 HTTP Message implementation
BSD 3-Clause "New" or "Revised" License
1.56k stars 152 forks source link

Updates reason phrase for 425 response status code #313

Closed weierophinney closed 6 years ago

weierophinney commented 6 years ago

The IANA HTTP status codes list was updated on 2018-07-02 to reflect a new IETF draft, draft-ietf-httpbis-replay-04, which details a header that may be used to indicate that an "early data" request may not yet be accepted, as no TLS handshake has been previously made, and that the request myust be retried after receiving the 425 response.

Without this change, tests fail on all platforms as the hard-coded reason phrase in Zend\Diactoros\Response references an early, non-standards-track usage of the status code.

samsonasik commented 6 years ago

@weierophinney should zend-http Response class be updated as well ? https://github.com/zendframework/zend-http/blob/master/src/Response.php#L143

weierophinney commented 6 years ago

@samsonasik Yes, please!

samsonasik commented 6 years ago

@weierophinney I've created PR https://github.com/zendframework/zend-http/pull/153 for it