Qpid
  1. Qpid
  2. QPID-3267

Java broker authentication does not work on JRE's other than Sun's

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 0.6, 0.7, 0.8, 0.9, 0.10, 0.11, 0.12
    • Fix Version/s: 0.13
    • Component/s: Java Broker
    • Labels:
      None

      Description

      This is due to use of com.sun.security.auth.UserPrincipal in ServerSession which is sun-specific.

      1. QPID-3267.patch
        2 kB
        Danushka Menikkumbura

        Issue Links

          Activity

          Hide
          Danushka Menikkumbura added a comment -

          As I understand we need to have an internal implementation of UserPrincipal to fix this issue. Please review and apply the attached patch.

          Thanks,
          Danushka

          Show
          Danushka Menikkumbura added a comment - As I understand we need to have an internal implementation of UserPrincipal to fix this issue. Please review and apply the attached patch. Thanks, Danushka
          Hide
          Keith Wall added a comment -

          Hi Danushka

          I agree with the intention of the patch, but is there a reason why you didn't use the existing class UsernamePrincipal (org.apache.qpid.server.auth.sasl) rather than introduce a new?

          cheers Keith.

          Show
          Keith Wall added a comment - Hi Danushka I agree with the intention of the patch, but is there a reason why you didn't use the existing class UsernamePrincipal (org.apache.qpid.server.auth.sasl) rather than introduce a new? cheers Keith.
          Hide
          Robbie Gemmell added a comment -

          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.

          Show
          Robbie Gemmell added a comment - 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 .
          Hide
          Robbie Gemmell added a comment -

          Hi Danushka,

          With the patch for QPID-3310 now applied, this issue should hopefully be resolved, so I've pushed this JIRA to Ready To Review as a result, Can you check the updated code meets your need and close this out if it does?

          Thanks,
          Robbie

          Show
          Robbie Gemmell added a comment - Hi Danushka, With the patch for QPID-3310 now applied, this issue should hopefully be resolved, so I've pushed this JIRA to Ready To Review as a result, Can you check the updated code meets your need and close this out if it does? Thanks, Robbie
          Hide
          Robbie Gemmell added a comment -

          Review-OK'ing issue as part of JIRA cleanup. Issue could be resolved, but may not be: see QPID-3469 for further details.

          Show
          Robbie Gemmell added a comment - Review-OK'ing issue as part of JIRA cleanup. Issue could be resolved, but may not be: see QPID-3469 for further details.
          Hide
          Robbie Gemmell added a comment -

          Updating this since it looks to be complete, marking for 0.13 instead of JIRA cleanup from above comment/action.

          Show
          Robbie Gemmell added a comment - Updating this since it looks to be complete, marking for 0.13 instead of JIRA cleanup from above comment/action.

            People

            • Assignee:
              Unassigned
              Reporter:
              Danushka Menikkumbura
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development