Working on #278425-55: Using basename() is not locale safe
Details http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2
protected function curlHeaderCallback($curlHandler, $header) {
- $this->headers[] = $header;
+ // Headers could be splited on chunks so prepending last header line if
+ // current header has no colon (":"). Additional details could be found
+ // in rfc2616 http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2
+ if (strpos($header, ':') === FALSE) {
+ $this->headers[] = array_pop($this->headers) . $header;
+ }
+ else {
+ $this->headers[] = $header;
+ }
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 869408-simpletest-headers.patch | 1.13 KB | andypost |
| #3 | 869408-simpletest-headers.patch | 1.17 KB | andypost |
| #2 | 869408-simpletest-headers.patch | 929 bytes | andypost |
| #1 | 869408-simpletest-headers.patch | 895 bytes | andypost |
Comments
Comment #1
andypostPatch
Comment #2
andypostThere should special check for HTTP/ header
strpos($header, 'HTTP/') !== 0Comment #3
andypostWe need detect
- start of headers "HTTP/"
- end of headers ""
- chunks "lines without colons
Comment #4
andypostSuppose it's a last patch
Comment #5
boombatower commentedComment #6
andypostThere's a patch already for core D7 #278425-66: Using basename() is not locale safe
This bug should be fixed in simpletest module also
Comment #7
boombatower commented#890440: Backport latest SimpleTest code from D7