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

Eugene Koontz commented on ZOOKEEPER-1195:
------------------------------------------

Hi Tom, 
 I tried your tests; they look good to me, but you need to submit this as a 
patch to the existing source tree. I can show you how to do this using Git if 
you want. Also I think it would be better to add these tests to 
https://issues.apache.org/jira/browse/ZOOKEEPER-1196.

> SASL authorizedID being incorrectly set: should use getHostName() rather than 
> getServiceName()
> ----------------------------------------------------------------------------------------------
>
>                 Key: ZOOKEEPER-1195
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1195
>             Project: ZooKeeper
>          Issue Type: Bug
>    Affects Versions: 3.4.0
>            Reporter: Eugene Koontz
>            Assignee: Eugene Koontz
>             Fix For: 3.4.0
>
>         Attachments: SaslAuthNamingTest.java, ZOOKEEPER-1195.patch
>
>
> Tom Klonikowski writes:
>     Hello developers,
>     the SaslServerCallbackHandler in trunk changes the principal name
>     service/host@REALM to service/service@REALM (i guess unintentionally).
>     lines 131-133:
>     if (!removeHost() && (kerberosName.getHostName() != null)) {
>       userName += "/" + kerberosName.getServiceName();
>     }
>     Server Log:
>     SaslServerCallbackHandler@115] - Successfully authenticated client:
>     authenticationID=fetcher/ubook@QUINZOO;
>     authorizationID=fetcher/ubook@QUINZOO.
>     SaslServerCallbackHandler@137] - Setting authorizedID:
>     fetcher/fetcher@QUINZOO

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to