mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Toenshoff" <>
Subject Re: Review Request 27494: Integrated CRAM-MD5 Test Authenticatee module into slave.
Date Mon, 10 Nov 2014 14:24:56 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Nov. 10, 2014, 2:24 p.m.)

Review request for mesos, Adam B and Vinod Kone.


Now following non optional Credential parameter Authenticatee interface.

Bugs: MESOS-2040

Repository: mesos-git


Also fixes messages.proto to use a raw bytestream instead of a string for AuthenticationStartMessage
as non CRAM-MD5 authentication may transmit binary data.
Note that the change of AuthenticationStartMessage does basically have no impact on C++ based
proto code other than the prevention of a warning due to non-UTF8 characters being encoded.
That does in fact occur when using non CRAM-MD5 based SASL authentication mechanisms.

Note that this patch covers modularized slave authentication only. Framework authentication
is currently covered by the default (built-in) implementation. There will be a subsequent
patch for modularized framework authentication. 

Diffs (updated)

  src/messages/messages.proto de0e2a2 
  src/sched/sched.cpp 8ca0526 
  src/scheduler/scheduler.cpp c74187c 
  src/slave/constants.hpp 701dd89 
  src/slave/constants.cpp d6ad78c 
  src/slave/flags.hpp efbd35d 
  src/slave/slave.hpp 72bbec9 
  src/slave/slave.cpp 81e0c4b 



make check

NOTE all three CRAM-MD5 authenticatee module related RRs need to get applied before running
make check.


Till Toenshoff

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message