Re: Review Request 57298: Added a new libprocess test for invalid principals.

2017-03-03 Thread Greg Mann
> On March 4, 2017, 12:35 a.m., Vinod Kone wrote: > > 3rdparty/libprocess/src/tests/http_tests.cpp > > Lines 1873 (patched) > > > > > > should we `delete` this constructor as a way to enforce? ofcourse one > >

Re: Review Request 57298: Added a new libprocess test for invalid principals.

2017-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57298/ --- (Updated March 4, 2017, 1:36 a.m.) Review request for mesos, Alexander Rojas,

Re: Review Request 57298: Added a new libprocess test for invalid principals.

2017-03-03 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57298/#review167910 --- Ship it! 3rdparty/libprocess/src/tests/http_tests.cpp Lines

Review Request 57298: Added a new libprocess test for invalid principals.

2017-03-03 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57298/ --- Review request for mesos, Alexander Rojas, Jan Schlicht, and Vinod Kone. Bugs: