Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-12 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/#review132904 --- Ship it! Ship It! - Alexander Rojas On May 10, 2016, 10:30

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-10 Thread Greg Mann
> On May 9, 2016, 10:35 a.m., Alexander Rojas wrote: > > src/master/main.cpp, lines 430-431 > > > > > > Does it? The container `Option` is copied, but all copies of that > > `Option` will be refering to the same

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-10 Thread Greg Mann
> On May 9, 2016, 10:35 a.m., Alexander Rojas wrote: > > src/master/main.cpp, lines 430-431 > > > > > > Does it? The container `Option` is copied, but all copies of that > > `Option` will be refering to the same

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-10 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/#review132536 --- Ship it! Ship It! - Kapil Arya On May 10, 2016, 4:30 p.m.,

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/ --- (Updated May 10, 2016, 8:30 p.m.) Review request for mesos, Alexander Rojas

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/ --- (Updated May 10, 2016, 5:06 p.m.) Review request for mesos, Alexander Rojas

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-10 Thread Greg Mann
> On May 9, 2016, 10:35 a.m., Alexander Rojas wrote: > > src/master/main.cpp, lines 430-431 > > > > > > Does it? The container `Option` is copied, but all copies of that > > `Option` will be refering to the same

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-09 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/#review132240 --- src/common/http.cpp (line 587)

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-06 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/ --- (Updated May 6, 2016, 9:14 p.m.) Review request for mesos, Alexander Rojas and

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-03 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/#review131614 --- src/common/http.cpp (lines 587 - 588)

Re: Review Request 46867: Enabled authorization of libprocess HTTP endpoints (Mesos).

2016-05-01 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46867/ --- (Updated May 1, 2016, 10:43 p.m.) Review request for mesos, Alexander Rojas