ZOOKEEPER-2040: Server to log underlying cause of SASL connection pro…

…blems. (Steve Loughran via cnauroth) git-svn-id: https://svn.apache.org/repos/asf/zookeeper/trunk@1702378 13f79535-47bb-0310-9956-ffa450edef68
apache · Sep 11, 2015 · 35e4551 · 35e4551
1 parent b12e5a1
commit 35e4551
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -256,6 +256,9 @@ IMPROVEMENTS:
   ZOOKEEPER-2270: Allow MBeanRegistry to be overridden for better unit tests
   (Jordan Zimmerman via rgs)
 
+  ZOOKEEPER-2040: Server to log underlying cause of SASL connection problems.
+  (Steve Loughran via cnauroth)
+
 Release 3.5.0 - 8/4/2014
 
 NEW FEATURES:

diff --git a/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java b/src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java
@@ -1077,7 +1077,7 @@ private Record processSasl(ByteBuffer incomingBuffer, ServerCnxn cnxn) throws IO
                 }
             }
             catch (SaslException e) {
-                LOG.warn("Client failed to SASL authenticate: " + e);
+                LOG.warn("Client failed to SASL authenticate: " + e, e);
                 if ((System.getProperty("zookeeper.allowSaslFailedClients") != null)
                   &&
                   (System.getProperty("zookeeper.allowSaslFailedClients").equals("true"))) {