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?


---

Reply via email to