php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #73297 HTTP stream wrapper should ignore HTTP 100 Continue
Submitted: 2016-10-11 21:02 UTC Modified: 2016-11-17 11:38 UTC
From: rowan dot collins at gmail dot com Assigned: jpauli
Status: Closed Package: Streams related
PHP Version: 5.6 OS:
Private report: No CVE-ID:
 [2016-10-11 21:02 UTC] rowan dot collins at gmail dot com
Description:
------------
This bug report forms part of an attempt to test and fix the implementation of HTTP/1.1 in the HTTP stream wrapper. See http://marc.info/?l=php-internals&m=147544280019519&w=2

A "100 Continue" response is *usually* only sent in response to an "Expect: 100-continue" header, and *usually* only useful with requests which have a body (e.g. POST, PUT). As such, it is unlikely to be encountered by someone sending a GET request using e.g. file_get_contents, but this is not guaranteed.

According to RFC 7231 section 6.2:

> A client MUST be able to parse one or more 1xx responses received
> prior to a final response, even if the client does not expect one.  A
> user agent MAY ignore unexpected 1xx responses.

However, the code currently treats these statuses as errors:

> HTTP request failed! HTTP/1.1 100 Continue


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-10-11 21:05 UTC] rowan dot collins at gmail dot com
I've created a failing phpt test for this https://gist.github.com/IMSoP/b893785c9c8f04c2b5ce3b0a638b33a7 and intend to attempt a fix.
 [2016-10-23 20:16 UTC] rowan dot collins at gmail dot com
-PHP Version: master-Git-2016-10-11 (Git) +PHP Version: 5.6
 [2016-10-23 20:16 UTC] rowan dot collins at gmail dot com
Pull request filed against 5.6: https://github.com/php/php-src/pull/2175
 [2016-11-17 11:38 UTC] jpauli@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: jpauli
 [2016-11-17 11:38 UTC] jpauli@php.net
Thank you for your bug report. This issue has already been fixed
in the latest released version of PHP, which you can download at 
http://www.php.net/downloads.php

https://github.com/php/php-src/pull/2175 was merged
 
PHP Copyright © 2001-2017 The PHP Group
All rights reserved.
Last updated: Fri Jun 23 17:01:39 2017 UTC