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

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

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

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

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

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

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

2016-05-09 Thread Greg Mann
> On May 9, 2016, 10:28 a.m., Alexander Rojas wrote: > > 3rdparty/libprocess/include/process/http.hpp, line 85 > > > > > > Is there a design doc or an overall explanation of how authorization > > will be performed

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

2016-05-09 Thread Greg Mann
> On May 9, 2016, 10:28 a.m., Alexander Rojas wrote: > > 3rdparty/libprocess/src/process.cpp, line 3289 > > > > > > Why no setting this variable close to were it will be used. I always > > liked the concept of

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

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

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

2016-05-09 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46866/#review132237 --- 3rdparty/libprocess/include/process/http.hpp (line 85)

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

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

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

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

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

2016-05-04 Thread Greg Mann
> On May 4, 2016, 3:24 a.m., Kapil Arya wrote: > > 3rdparty/libprocess/include/process/http.hpp, lines 89-91 > > > > > > Is that the correct line-break here? Can we make it better? I read through the relevant

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

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

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

2016-05-03 Thread Kapil Arya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46866/#review131613 --- 3rdparty/libprocess/include/process/http.hpp (lines 89 - 91)

Review Request 46866: Enabled authorization of libprocess HTTP endpoints (libprocess).

2016-05-01 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46866/ --- Review request for mesos, Alexander Rojas and Kapil Arya. Bugs: MESOS-5286