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

Robbie Gemmell commented on QPID-3267:
--------------------------------------

I echo Keith's comments about the unecessary duplication, and would add that 
there was also no unit test for the new class in the patch (although that was 
also true of the existing class at the time, but I see Keith added tests in 
subsequent work he did).

Given that the majority of the attached patch is unecessary due to the above, 
and it looks like the patch in QPID-3310 removes the use of 
com.sun.security.auth.UserPrincipal in ServerSession anyway whilst to use 
Subjects, I am just going to incorporate this issue into QPID-3310.

> Java broker authentication does not work on JRE's other than Sun's
> ------------------------------------------------------------------
>
>                 Key: QPID-3267
>                 URL: https://issues.apache.org/jira/browse/QPID-3267
>             Project: Qpid
>          Issue Type: Bug
>            Reporter: Danushka Menikkumbura
>            Assignee: Robbie Gemmell
>            Priority: Critical
>         Attachments: QPID-3267.patch
>
>
> This is due to use of com.sun.security.auth.UserPrincipal in ServerSession 
> which is sun-specific.

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

        

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:[email protected]

Reply via email to