----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56754/#review168562 -----------------------------------------------------------
src/authentication/executor/jwt_secret_generator.cpp Lines 39 (patched) <https://reviews.apache.org/r/56754/#comment240805> i think we indent this by 2 spaces? src/authentication/executor/jwt_secret_generator.cpp Lines 48 (patched) <https://reviews.apache.org/r/56754/#comment240806> Should we return a `Failure` if `Principal.value` is `Some`? - Vinod Kone On March 8, 2017, 2:26 p.m., Jan Schlicht wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56754/ > ----------------------------------------------------------- > > (Updated March 8, 2017, 2:26 p.m.) > > > Review request for mesos, Alexander Rojas and Greg Mann. > > > Bugs: MESOS-7000 > https://issues.apache.org/jira/browse/MESOS-7000 > > > Repository: mesos > > > Description > ------- > > This can be used to create a 'Secret' from a 'Principal'. > The resulting secret will provide a JWT. > > > Diffs > ----- > > src/Makefile.am bb917c5d1b36f5106a74914fa3a864038a95e8bb > src/authentication/executor/jwt_secret_generator.hpp PRE-CREATION > src/authentication/executor/jwt_secret_generator.cpp PRE-CREATION > src/tests/secret_generator_tests.cpp PRE-CREATION > > > Diff: https://reviews.apache.org/r/56754/diff/8/ > > > Testing > ------- > > make check > > > Thanks, > > Jan Schlicht > >
