Bug #73882 [Asn->Csd]: Patch for php_stream_url_wrap_http_ex to support multiple spaces

From: 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

« previous php.bugs (#206388) next »