faeez/mxmbsocket
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Repo. Project Releases Wiki Activity

Remove usage of http_parse_message

Browse Source 
Found a case where http_parse_message failed to parse a valid request header and returned null, which ended up in an uncaught exception being thrown by Guzzle.
This commit is contained in:
Chris Boden 2017-09-13 18:12:50 -04:00
parent 0cde24bae7
commit fe4a97400d
1 changed files with 1 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/Ratchet/Http/HttpRequestParser.php
View File

@ -59,18 +59,6 @@ class HttpRequestParser implements MessageInterface {
* @return \Psr\Http\Message\RequestInterface
*/
public function parse($headers) {
if (function_exists('http_parse_message')) {
$parts = http_parse_message($headers);
return new gPsr\Request(
$parts->requestMethod
, $parts->requestUrl
, $parts->headers
, null
, $parts->httpVersion
);
} else {
return gPsr\parse_request($headers);
}
return gPsr\parse_request($headers);
}
}