Re: Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-13 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/ --- (Updated April 13, 2017, 6:05 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-12 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/ --- (Updated April 12, 2017, 4:40 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-12 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/#review171707 --- Ship it! Ship It! - Alexander Rojas On April 8, 2017, 1:15 a

Re: Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-10 Thread Adam B
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/#review171525 --- LGTM. Just get rid of the unnecessary parentheses. src/authorize

Re: Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-10 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/#review171478 --- Ship it! Ship It! - Vinod Kone On April 7, 2017, 11:15 p.m.,

Re: Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-07 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/ --- (Updated April 7, 2017, 11:15 p.m.) Review request for mesos, Adam B, Alexander

Re: Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-07 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/#review171317 --- src/authorizer/local/authorizer.cpp Line 1091 (original), 1091 (p

Review Request 58252: Allowed the local authorizer to accept subjects with no value.

2017-04-06 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58252/ --- Review request for mesos, Adam B, Alexander Rojas, Till Toenshoff, and Vinod Kon