----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48781/#review138069 -----------------------------------------------------------
include/mesos/authorizer/acls.proto (line 182) <https://reviews.apache.org/r/48781/#comment203278> why optional fields in state.json??? should be acls.proto or? - Joerg Schad On June 16, 2016, 10:06 a.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48781/ > ----------------------------------------------------------- > > (Updated June 16, 2016, 10:06 a.m.) > > > Review request for mesos, Joerg Schad and Till Toenshoff. > > > Bugs: MESOS-5588 > https://issues.apache.org/jira/browse/MESOS-5588 > > > Repository: mesos > > > Description > ------- > > The messages `GetEndpoints`, `ViewFramework`, `ViewTask`, `ViewExecutor` > and `AccessSandbox` all have optional authorization objects as a result > of copy and pasting previous message, but their semantics were those > of an required field, which led to some unexpected behavior when a user > misstyped any entry there. > > This patch sets the fields to their actual expected values. > > > Diffs > ----- > > include/mesos/authorizer/acls.proto > 2ffcaa29deaccba5e099a81706432035fe318d98 > > Diff: https://reviews.apache.org/r/48781/diff/ > > > Testing > ------- > > Ran variations of this script for both master and agent: > > ```sh > cat <<EOF > /tmp/credentials.txt > foo bar > baz bar > EOF > > cat <<EOF > /tmp/acls.json > { > "permissive": false, > "access_mesos_logs" : [ > { > "principals" : { "values" : ["foo"] }, > "logs" : { "type" : "ANY" } > } > ], > "view_frameworks": [ > { > "principals": { "type": "ANY" }, > "user": { "type": "NONE" } > } > ] > } > EOF > > ./bin/mesos-master.sh --work_dir=/tmp/mesos/master \ > --authenticate_http \ > --credentials=file:///tmp/credentials.txt \ > --acls=file:///tmp/acls.json \ > --log_dir=/tmp/mesos/logs/master > ``` > > with valid and invalid ACLs > > > Thanks, > > Alexander Rojas > >