Qpid
  1. Qpid
  2. QPID-4142

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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

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

        I approve this fix going into 0.18

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

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

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

        I also approve this fix for 0.18.

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

        Chuck merged this to the release branch at revision 1372597.

        Show
        Justin Ross added a comment - - edited Chuck merged this to the release branch at revision 1372597.
        Justin Ross made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Chuck Rolke [ chug ]
        Fix Version/s 0.18 [ 12322451 ]
        Resolution Fixed [ 1 ]
        Hide
        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
        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.
        Chuck Rolke made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development