Uploaded image for project: 'Qpid'
  1. Qpid
  2. QPID-4142

C++ broker connection counting username error when used in ha cluster and auth is EXTERNAL

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.18
    • Fix Version/s: 0.18
    • Component/s: C++ Broker
    • Labels:
      None

      Description

      In a cluster setup with auth enabled and set to EXTERNAL then the Acl connection counting does not get the correct username associated with a shadowed connection.

      The cluster member that accepts the connection accounts for the correct user name. Other cluster members that receive shadow connections see user name 'anonymous'.

        Activity

        Hide
        chug Chuck Rolke added a comment -
        Show
        chug Chuck Rolke added a comment - http://svn.apache.org/viewvc?view=revision&revision=1371772 is the fix for this issue.
        Hide
        astitcher Andrew Stitcher added a comment -

        I approve this fix going into 0.18

        Show
        astitcher Andrew Stitcher added a comment - I approve this fix going into 0.18
        Hide
        justi9 Justin Ross added a comment -

        Reviewed by Andrew (and Alan, as it happens). Approved for 0.18.

        Show
        justi9 Justin Ross added a comment - Reviewed by Andrew (and Alan, as it happens). Approved for 0.18.
        Hide
        aconway Alan Conway added a comment -

        I also approve this fix for 0.18.

        Show
        aconway Alan Conway added a comment - I also approve this fix for 0.18.
        Hide
        justi9 Justin Ross added a comment - - edited

        Chuck merged this to the release branch at revision 1372597.

        Show
        justi9 Justin Ross added a comment - - edited Chuck merged this to the release branch at revision 1372597.
        Hide
        chug Chuck Rolke added a comment -

        This patch was for 0.18 only. Since it has been merged to the 0.18 branch it has no business on trunk. Checkin r1382155 reverts the change in trunk.

        Show
        chug Chuck Rolke added a comment - This patch was for 0.18 only. Since it has been merged to the 0.18 branch it has no business on trunk. Checkin r1382155 reverts the change in trunk.

          People

          • Assignee:
            chug Chuck Rolke
            Reporter:
            chug Chuck Rolke
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development