-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2252/
-----------------------------------------------------------

Review request for zookeeper and Eugene Koontz.


Summary
-------

It now supports DIGEST-MD5 and GSSAPI (and virtually any mech SASL supports). 
cli_mt works with GSSAPI, cli_st with DIGEST-MD5 and GSSAPI

Example: cli_st -u super -h zk-sasl-md5 -m DIGEST-MD5 localhost:2181
         cli_mt -M GSSAPI localhost:2181

* requires libsasl2 and plugins

* zookeeper
  o zoo_(a)sasl(*zh, *conn, *data, len, cptr)
* zk_sasl
  o zoo_sasl_init(*callbacks)
  o zoo_sasl_connect(*zh, *servicename, *host, **conn, **supportedmechs, 
*mechcount)
  o zoo_(a)sasl_authenticate(*zh, *conn, *mech, *supportedmechs)

This latest patch includes

* detection of libsasl2
* configure option --with-sasl (default: on)
* compiler flag SASL
* a simple test for DIGEST-MD5 authentication


This addresses bug 1112.
    https://issues.apache.org/jira/browse/1112


Diffs
-----

  src/c/src/zk_adaptor.h 6aed38f 
  src/c/src/zk_sasl.h PRE-CREATION 
  src/c/src/zk_sasl.c PRE-CREATION 
  src/c/src/cli.c c9fe4dd 
  src/c/Makefile.am dbf4080 
  src/c/configure.ac 4e1a8d5 
  src/c/include/proto.h bce408e 
  src/c/include/zookeeper.h 7d1066a 
  src/c/src/zookeeper.c 64d7a4d 
  src/c/tests/TestClient.cc 2d45cec 
  src/c/tests/jaas.digest.server.conf PRE-CREATION 
  src/c/tests/zkServer.sh 0af80f8 
  src/java/main/org/apache/zookeeper/server/ZooKeeperSaslServer.java 0ca18dc 

Diff: https://reviews.apache.org/r/2252/diff


Testing
-------


Thanks,

Tom

Reply via email to