mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gilbert Song <>
Subject Re: Review Request 48917: Fixed docker fetcher 3xx redirect errors by header attached.
Date Thu, 13 Apr 2017 05:33:10 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated April 12, 2017, 10:33 p.m.)

Review request for mesos, Avinash sridharan, Chun-Hung Hsiao, Artem Harutyunyan, Ilya Pronin,
Jie Yu, Timothy Chen, and Vinod Kone.

Bugs: MESOS-5172

Repository: mesos


The root cause for this issue is that, in private registry
like, layer download request will be redirected to
storage server in S3. However, the curl command with '-L'
handles HTTP redirection automatically, in which case HTTP
headers will be attached to all requests. AmazonS3 server
will return 400 Bad Request if HTTP Authorization header
is attached, with 'InvalidArgument' error code. So we need
to touch the given URL first to add extra logic for HTTP

Please note that the download() method is changed to be
recursive since no header should be attached once the
request get authenticated.

Diffs (updated)

  src/uri/fetchers/docker.cpp d6d2e8e72554c5f96c3875af8ef93e0bc2107af6 




make check

Tested with manually.


Gilbert Song

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message