You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Ashwin Murthy <as...@gmail.com> on 2016/03/18 00:44:13 UTC

Review Request 44971: MESOS-3902: Fix location header in redirect from non-leading master.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44971/
-----------------------------------------------------------

Review request for mesos and Vinod Kone.


Repository: mesos


Description
-------

MESOS-3902: Fix location header in redirect from non-leading master. Added the scheme and path to the URL that was being set in the location header


Diffs
-----

  src/master/http.cpp 950206baf7f3a1cccdc49d810126473966d8d021 

Diff: https://reviews.apache.org/r/44971/diff/


Testing
-------

None yet. Was attempting to write a test but turns there is no good way to test multi master in our testing framework. Will need to test this manually by launching multiple masters with ZK setup. Doing that in parallel


Thanks,

Ashwin Murthy