Re: Review Request 56901: Updated master validation code to use 'AuthenticationContext'.

2017-02-27 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56901/#review166862 --- Ship it! Ship It! - Alexander Rojas On Feb. 22, 2017, 8:48

Re: Review Request 56901: Updated master validation code to use 'AuthenticationContext'.

2017-02-24 Thread Greg Mann
> On Feb. 24, 2017, 1:53 a.m., Vinod Kone wrote: > > src/master/validation.cpp, line 339 > > > > > > Should this be a CHECK? When is this possible? We'll soon be adding an authentication module (the JWT

Re: Review Request 56901: Updated master validation code to use 'AuthenticationContext'.

2017-02-24 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56901/#review166737 --- src/master/validation.cpp (line 339)

Re: Review Request 56901: Updated master validation code to use 'AuthenticationContext'.

2017-02-23 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56901/#review166633 --- src/master/validation.cpp (line 339)

Re: Review Request 56901: Updated master validation code to use 'AuthenticationContext'.

2017-02-22 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56901/ --- (Updated Feb. 22, 2017, 7:48 p.m.) Review request for mesos, Adam B, Alexander

Review Request 56901: Updated master validation code to use 'AuthenticationContext'.

2017-02-21 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56901/ --- Review request for mesos, Adam B, Alexander Rojas, Jan Schlicht, Till