----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3631/#review4606 -----------------------------------------------------------
src/java/main/org/apache/zookeeper/ClientCnxn.java <https://reviews.apache.org/r/3631/#comment10253> SASL connection also needs to be attempted when ZooKeeperSaslClient.LOGIN_CONTEXT_NAME_KEY is set as system property and java.security.auth.login.config is not set (that's what happens with Hadoop security). That was part of my patch. - Thomas On 2012-01-25 18:56:42, Eugene Koontz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3631/ > ----------------------------------------------------------- > > (Updated 2012-01-25 18:56:42) > > > Review request for zookeeper. > > > Summary > ------- > > ZOOKEEPER-1373: Hardcoded SASL login context name clashes with Hadoop > security configuration override > > Fix is to allow system property to designate the JAAS configuration section > that the zookeeper client will use. > > > This addresses bug ZOOKEEPER-1373. > https://issues.apache.org/jira/browse/ZOOKEEPER-1373 > > > Diffs > ----- > > conf/zoo_sample.cfg aafb324 > src/java/main/org/apache/zookeeper/ClientCnxn.java 6c25e40 > src/java/main/org/apache/zookeeper/client/ZooKeeperSaslClient.java 722538e > src/java/test/org/apache/zookeeper/test/SaslAuthDesignatedClientTest.java > PRE-CREATION > > src/java/test/org/apache/zookeeper/test/SaslAuthFailDesignatedClientTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/3631/diff > > > Testing > ------- > > "ant test" java tests pass. > > > Thanks, > > Eugene > >
