> On Jan. 25, 2019, 4:01 p.m., Vinod Kone wrote: > > src/scheduler/scheduler.cpp > > Line 234 (original), 234 (patched) > > <https://reviews.apache.org/r/69839/diff/1/?file=2122187#file2122187line234> > > > > s/Augmenting/Authenticating/ > > Alexander Rukletsov wrote: > Please no: authenticatee does not really authenticate, but adds > authentication-related headers to the request.
Oh right. I remember that. How about "Adding authentication headers to ..." or something like that? "Augmenting ..." is very opaque and doesn't convey any useful information for someone looking at logs. - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69839/#review212325 ----------------------------------------------------------- On Jan. 25, 2019, 3:41 p.m., Till Toenshoff wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69839/ > ----------------------------------------------------------- > > (Updated Jan. 25, 2019, 3:41 p.m.) > > > Review request for mesos, Alexander Rukletsov and Vinod Kone. > > > Bugs: MESOS-9210 > https://issues.apache.org/jira/browse/MESOS-9210 > > > Repository: mesos > > > Description > ------- > > The HTTP scheduler API dictates that on a single connection, the scheduler may > only send a single SUBSCRIBE request. Due to recent authentication related > changes, this contract got broken. This patch restores the contract and adds a > test validating that the library is enforcing it. > > > Diffs > ----- > > src/scheduler/scheduler.cpp cb24ba9c8e1d04b8c62bdf07b12758a61b3bf036 > src/tests/scheduler_tests.cpp b571bb1d20744b943580677a26db4c12c7c311d1 > > > Diff: https://reviews.apache.org/r/69839/diff/1/ > > > Testing > ------- > > manual testing; > Running the included test without patching `scheduler.cpp` -> fails as the > master does in fact receive two SUBSCRIBE requests. > > `make check` > > > Thanks, > > Till Toenshoff > >
