zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From abhishek-chouhan <...@git.apache.org>
Subject [GitHub] zookeeper issue #541: ZOOKEEPER-3059 EventThread leak in case of Sasl AuthFa...
Date Fri, 22 Jun 2018 14:22:33 GMT
Github user abhishek-chouhan commented on the issue:

    https://github.com/apache/zookeeper/pull/541
  
    @anmolnar  The second one happens when zooKeeperSaslClient.initialize(ClientCnxn.this)
throws an sasl exception, which from the code looks like can happen in case of incomplete
initialization. In zookeeperSaslClient's constructor we call createSaslClient which in case
of anything apart from LoginException will swallow the exception and would initialize the
saslClient object as null(which would cause initialize to throw sasl exception in the null
check). It however looks hard to simulate an exception in the initialization part, since there
are static methods involved and stubbing zookeepersaslclient itself doesn't look to be an
option.


---

Mime
View raw message