Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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/') !== 0
Comment #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 CreditAttribution: 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 CreditAttribution: boombatower commented#890440: Backport latest SimpleTest code from D7