-----------------------------------------------------------
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
> 
>

Reply via email to