zookeeper-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [zookeeper] eolivelli commented on a change in pull request #118: ZOOKEEPER-1634: hardening security by teaching server to enforce client authentication.
Date Mon, 17 Jun 2019 05:33:28 GMT
eolivelli commented on a change in pull request #118: ZOOKEEPER-1634: hardening security by
teaching server to enforce client authentication.
URL: https://github.com/apache/zookeeper/pull/118#discussion_r294139936
 
 

 ##########
 File path: zookeeper-server/src/main/java/org/apache/zookeeper/client/ZooKeeperSaslClient.java
 ##########
 @@ -136,6 +138,8 @@ public ZooKeeperSaslClient(final String serverPrincipal, ZKClientConfig
clientCo
             // configuration isn't set. We can reevaluate whether to catch RuntimeException
instead when more 
             // different types of RuntimeException found
             runtimeException = e;
+        } catch (Exception e) {
+            LOG.info("shit {}", e);
 
 Review comment:
   Drop this debug and bad word

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message