Patrick Hunt commented on ZOOKEEPER-844:

IIRC qabot will apply the most recent patch, it only knows about trunk though 
(no branches).

The easiest way to address is to attach the patch for br33 first, then attach 
the patch for trunk. Pretty hacky but it works ok for me.

The other option is to just attach both patches and state in a comment the 
problem, ask a committer to review it manually instead of waiting on qabot.


> handle auth failure in java client
> ----------------------------------
>                 Key: ZOOKEEPER-844
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-844
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: java client
>    Affects Versions: 3.3.1
>            Reporter: Camille Fournier
>            Assignee: Camille Fournier
>             Fix For: 3.3.2, 3.4.0
>         Attachments: ZOOKEEPER-844.patch
> ClientCnxn.java currently has the following code:
>   if (replyHdr.getXid() == -4) {
>                 // -2 is the xid for AuthPacket
>                 // TODO: process AuthPacket here
>                 if (LOG.isDebugEnabled()) {
>                     LOG.debug("Got auth sessionid:0x"
>                             + Long.toHexString(sessionId));
>                 }
>                 return;
>             }
> Auth failures appear to cause the server to disconnect but the client never 
> gets a proper state change or notification that auth has failed, which makes 
> handling this scenario very difficult as it causes the client to go into a 
> loop of sending bad auth, getting disconnected, trying to reconnect, sending 
> bad auth again, over and over. 

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

Reply via email to