----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2252/#review2438 -----------------------------------------------------------
src/c/include/zookeeper.h <https://reviews.apache.org/r/2252/#comment5542> I'm gonna replace the callback with two out parameters here. - Tom On 2011-10-07 18:58:28, Tom Klonikowski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2252/ > ----------------------------------------------------------- > > (Updated 2011-10-07 18:58:28) > > > Review request for zookeeper, Eugene Koontz, Mahadev Konar, and Tom > Klonikowski. > > > Summary > ------- > > This first patch implements the zookeeper sasl operations, extends the > zkServer.sh test script with the ability to start a sasl enabled server and > adds a test that checks the initial DIGEST-MD5 response. > > It introduces no external requirements but provides zoo_sasl/zoo_asasl > functions that allow applications (or the addon from patch #2) to communicate > with the sasl server backend. > > Patch #2 will add a simple api for sasl authentication, patch #3 includes a > sasl enabled command line client. > > > This addresses bug ZOOKEEPER-1112. > https://issues.apache.org/jira/browse/ZOOKEEPER-1112 > > > Diffs > ----- > > src/c/include/proto.h bce408e > src/c/include/zookeeper.h 7d1066a > src/c/src/zk_adaptor.h 6aed38f > src/c/src/zookeeper.c 64d7a4d > src/c/tests/TestClient.cc 2d45cec > src/c/tests/jaas.digest.server.conf e69de29 > src/c/tests/zkServer.sh 0af80f8 > > Diff: https://reviews.apache.org/r/2252/diff > > > Testing > ------- > > > Thanks, > > Tom > >