Bug #73882 [Asn->Csd]: Patch for php_stream_url_wrap_http_ex to support multiple spaces
| From: | nikic@php.net | Date: | Sun, 08 Jan 2017 10:56:37 +0000 |
| Subject: | Bug #73882 [Asn->Csd]: Patch for php_stream_url_wrap_http_ex to support multiple spaces | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-206388@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=73882&edit=1
ID: 73882
Updated by: nikic@php.net
Reported by: laurentconstantin at free dot fr
Summary: Patch for php_stream_url_wrap_http_ex to support
multiple spaces
-Status: Assigned
+Status: Closed
Type: Bug
Package: HTTP related
PHP Version: 5.6.29
Assigned To: nikic
Block user comment: N
Private report: N
New Comment:
I've merged PR https://github.com/php/php-src/pull/1902
yesterday, which is a fix for the same issue, which has been hanging around in the queue for some
months. As the PR is older and a bit more complete (it also removes trailing whitespace, which the
spec also allows), I went with it. But in any case, thanks for providing a patch :)
Previous Comments:
------------------------------------------------------------------------
[2017-01-06 15:51:41] laurentconstantin at free dot fr
Description:
------------
Hello,
The function php_stream_url_wrap_http_ex() in file ext/standard/http_fopen_wrapper.c
checks the presence of the "Transfer-Encoding: chunked" HTTP header to detect if
the filter has to be used.
However, the HTTP RFC allows multiple spaces after the ':'.
This patch adds the supports of this case.
This patch also updates the Location, Content-Type and Content-Length processing
which suffers the same bug. I've created two generic functions easier to use.
Thank you.
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=73882&edit=1