-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69075/
-----------------------------------------------------------
(Updated Oct. 22, 2018, 11:22 a.m.)
Review request for mesos, Alexander Rukletsov, Armand Grillet, Jie Yu, and
James Peach.
Changes
-------
Make HTTP 1.1 default but allow for fallback on HTTP 1.0 when curl does not
support otherwise.
Summary (updated)
-----------------
Updated docker image fetcher to enforce HTTP 1.x.
Bugs: MESOS-8907
https://issues.apache.org/jira/browse/MESOS-8907
Repository: mesos
Description (updated)
-------
Modifies the 'curl' invocation that is returning an http::Response,
locking it into HTTP 1.x. Our current HTTP parser is unable to process
HTTP 2 responses.
With the advent of curl 7.47, HTTPS connections are being enforced
towards HTTP 2 rather aggressively. As a result, our image fetcher
fails when recent curl versions are being used for pulling images from
a registry that supports HTTP 2.
HTTP 1.1 is chosen as long as the underlying curl supports the
'--http1.1' flag. If curl does not support that flag, fall back to
using HTTP 1.0.
For allowing all the benefits of HTTP 2 where possible, we do not
adapt any 'curl' invocations that do not attempt to parse headers.
Diffs (updated)
-----
src/uri/fetchers/docker.cpp 6b1277f822dcf80d44daa8133b3ecc8c9a34ef07
Diff: https://reviews.apache.org/r/69075/diff/3/
Changes: https://reviews.apache.org/r/69075/diff/2-3/
Testing
-------
`make check`
`sudo ./bin/mesos-tests.sh
--gtest_filter="ImageAlpine/ProvisionerDockerTest.ROOT_INTERNET_CURL_SimpleCommand/2"`
on a system with curl 7.59.0 installed.
Thanks,
Till Toenshoff
