----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45000/#review125434 -----------------------------------------------------------
Ship it! Thanks for following up with a real test! Btw, no need for that NOTE as the bug is attached to this review and the commit will be pasted to that ticket. I'll remove it while committing. - Vinod Kone On March 24, 2016, 2:24 a.m., Ashwin Murthy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45000/ > ----------------------------------------------------------- > > (Updated March 24, 2016, 2:24 a.m.) > > > Review request for mesos and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > MESOS-3902: [Final] Fix location header in redirect from non-leader. Fixed to > append the path from the original URL. > > > Diffs > ----- > > src/master/http.cpp 97e4b0ce3286540788e3e6c5b484687b83e000f6 > > Diff: https://reviews.apache.org/r/45000/diff/ > > > Testing > ------- > > This has been tested end to end with 3 local mesos masters and zk. Valided > the location header is now correct. > > > Thanks, > > Ashwin Murthy > >
