@katebutler opened this Pull Request on May 15th 2019 Member

Fix two issues observed when downloading the GeoIP database via cURL with HTTP/2:

  • The response headers are returned in lowercase e.g. 'content-length' instead of 'Content-Length'
  • Work around an apparent bug in libcurl where sending a Range header via CURLOPT_HTTPHEADER causes curl to error "Failed sending HTTP request". Sending it via CURLOPT_RANGE instead has been tested with HTTP/1.1 and HTTP/2 and works with both.

Fixes #14339

@Findus23 commented on May 15th 2019 Member

Perfect, this fixes it for me!
There might be more HTTP/2 bugs in Matomo, but at least this download now works fine.

This Pull Request was closed on July 16th 2019
Powered by GitHub Issue Mirror