mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jan Schlicht <>
Subject Re: Review Request 55496: Fixed parsing of proxy CONNECT responses.
Date Fri, 13 Jan 2017 15:51:39 GMT

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

(Updated Jan. 13, 2017, 4:51 p.m.)

Review request for mesos, Benjamin Bannier, Gilbert Song, and Jie Yu.


Addressed issues.

Bugs: MESOS-6010

Repository: mesos


When behind a proxy, access to websites using SSL can be done by using
HTTP CONNECT tunneling. This is, for example, used when we pull images
from a Docker repository. The HTTP response of such a CONNECT request
surfaced a problem with our HTTP parser, because it doesn't have neither
any headers nor a message body. The parser expected a 'Content-Length'
header to be set in every HTTP response. We fix this by assuming that
a HTTP response doesn't have a message body when the 'Content-Length'
header isn't set.

Diffs (updated)

  3rdparty/libprocess/src/decoder.hpp 4c779d42548958e610142438a57529ccb4478053 
  3rdparty/libprocess/src/tests/decoder_tests.cpp 87563f4fcdfaa2a33c4482533ddff24e062e603a



make check


Jan Schlicht

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