You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zookeeper.apache.org by fp...@apache.org on 2016/10/17 08:44:17 UTC

zookeeper git commit: ZOOKEEPER-2606: SaslServerCallbackHandler#handleAuthorizeCallback() should log the exception (Ted Yu via fpj)

Repository: zookeeper
Updated Branches:
  refs/heads/branch-3.4 a601b0329 -> 967c3a71b


ZOOKEEPER-2606: SaslServerCallbackHandler#handleAuthorizeCallback() should log the exception (Ted Yu via fpj)


Project: http://git-wip-us.apache.org/repos/asf/zookeeper/repo
Commit: http://git-wip-us.apache.org/repos/asf/zookeeper/commit/967c3a71
Tree: http://git-wip-us.apache.org/repos/asf/zookeeper/tree/967c3a71
Diff: http://git-wip-us.apache.org/repos/asf/zookeeper/diff/967c3a71

Branch: refs/heads/branch-3.4
Commit: 967c3a71bd8eaf1ac29b2702173115976874bd8e
Parents: a601b03
Author: fpj <fp...@apache.org>
Authored: Mon Oct 17 09:43:25 2016 +0100
Committer: fpj <fp...@apache.org>
Committed: Mon Oct 17 09:43:25 2016 +0100

----------------------------------------------------------------------
 CHANGES.txt                                                       | 3 +++
 .../apache/zookeeper/server/auth/SaslServerCallbackHandler.java   | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zookeeper/blob/967c3a71/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index f66378d..2b69758 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -28,6 +28,9 @@ IMPROVEMENTS:
   ZOOKEEPER-2594: Use TLS for downloading artifacts during build
   (Olaf Flebbe via phunt)
 
+  ZOOKEEPER-2606: SaslServerCallbackHandler#handleAuthorizeCallback() should
+  log the exception (Ted Yu via fpj)
+
 Release 3.4.9 - 2016-08-23
 
 Backward compatible changes:

http://git-wip-us.apache.org/repos/asf/zookeeper/blob/967c3a71/src/java/main/org/apache/zookeeper/server/auth/SaslServerCallbackHandler.java
----------------------------------------------------------------------
diff --git a/src/java/main/org/apache/zookeeper/server/auth/SaslServerCallbackHandler.java b/src/java/main/org/apache/zookeeper/server/auth/SaslServerCallbackHandler.java
index 2fbd6ed..7fdffde 100644
--- a/src/java/main/org/apache/zookeeper/server/auth/SaslServerCallbackHandler.java
+++ b/src/java/main/org/apache/zookeeper/server/auth/SaslServerCallbackHandler.java
@@ -134,7 +134,7 @@ public class SaslServerCallbackHandler implements CallbackHandler {
             LOG.info("Setting authorizedID: " + userNameBuilder);
             ac.setAuthorizedID(userNameBuilder.toString());
         } catch (IOException e) {
-            LOG.error("Failed to set name based on Kerberos authentication rules.");
+            LOG.error("Failed to set name based on Kerberos authentication rules.", e);
         }
     }