[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-1554?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13468619#comment-13468619
 ] 

Guillaume Nodet commented on ZOOKEEPER-1554:
--------------------------------------------

It looks like the following code in ZooKeeperSaslClient is wrong:

{code}
    public boolean clientTunneledAuthenticationInProgress() {
        // TODO: Rather than checking a disjunction here, should be a single 
member
        // variable or method in this class to determine whether the client is
        // configured to use SASL. (see also ZOOKEEPER-1455).
        try {
            if ((System.getProperty(Environment.JAAS_CONF_KEY) != null) ||
                (javax.security.auth.login.Configuration.getConfiguration() != 
null)) {
                // Client is configured to use SASL.
{code}

The reason is that javax.security.auth.login.Configuration.getConfiguration() 
is always non null.

                
> Can't use zookeeper client without SASL
> ---------------------------------------
>
>                 Key: ZOOKEEPER-1554
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1554
>             Project: ZooKeeper
>          Issue Type: Bug
>    Affects Versions: 3.4.4
>            Reporter: Guillaume Nodet
>            Priority: Blocker
>
> The ZooKeeperSaslClient correctly detects that it should not use SASL when 
> nothing is configured, however the SendThread waits forever because 
> clientTunneledAuthenticationInProgress() returns true instead of false.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to