[ https://issues.apache.org/jira/browse/ZOOKEEPER-1437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13397975#comment-13397975 ]
Patrick Hunt commented on ZOOKEEPER-1437: ----------------------------------------- I looked through the patch, some questions. 1) moving the xid is a bit concerning. Isn't there a chance that the next xid (A) will be generated, and a separate xid (B) generated, and B sent on the wire before A? Notice that getXid can be called by doio, but also by sendPacket via the event thread. Or does something else protect against this? 2) I'm still concerned about spinning on the write interest flag. Have you considered this? I'm still looking, but those were my initial thoughts. > Client uses session before SASL authentication complete > ------------------------------------------------------- > > Key: ZOOKEEPER-1437 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1437 > Project: ZooKeeper > Issue Type: Bug > Components: java client > Affects Versions: 3.4.3 > Reporter: Thomas Weise > Assignee: Eugene Koontz > Fix For: 3.4.4, 3.5.0 > > Attachments: ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch, > ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch, > ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch, > ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch, > ZOOKEEPER-1437.patch, ZOOKEEPER-1437.patch > > > Found issue in the context of hbase region server startup, but can be > reproduced w/ zkCli alone. > getData may occur prior to SaslAuthenticated and fail with NoAuth. This is > not expected behavior when the client is configured to use SASL. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira