Github user anmolnar commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/496#discussion_r177697912
--- Diff:
src/java/main/org/apache/zookeeper/server/quorum/auth/SaslQuorumAuthLearner.java
---
@@ -134,6 +138,8 @@ public void authenticate(Socket sock, String hostName)
throws IOException {
// Validate status code at the end of authentication exchange.
checkAuthStatus(sock, qpStatus);
+ } catch (RuntimeException e) {
--- End diff --
@brettKK What's the point of swallowing it here?
---