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

QMF console.py does not correctly set authorized user-id

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.19
    • Fix Version/s: 0.19
    • Component/s: Python Client, QMF
    • Labels:
      None

      Description

      There appears to be a regression that prevents console.py (python QMF client) from correctly setting the authenticated user-id in outbound messages.

      This will cause authentication errors, although the correct credentials have been supplied.

        Activity

        Show
        kgiusti Ken Giusti added a comment - http://svn.apache.org/viewvc?view=revision&revision=1372567
        Hide
        kgiusti Ken Giusti added a comment -

        It appears as if the format of the user identifier available in the connection has changed from a "tuple" type to a string.

        The following patch works around this:

        Index: console.py
        ===================================================================
        — console.py (revision 1372526)
        +++ console.py (working copy)
        @@ -2443,6 +2443,8 @@
        uid = self.conn.user_id
        if uid._class_ == tuple and len(uid) == 2:
        self.saslUser = uid[1]
        + elif type(uid) is str:
        + self.saslUser = uid;
        else:
        self.saslUser = None

        Show
        kgiusti Ken Giusti added a comment - It appears as if the format of the user identifier available in the connection has changed from a "tuple" type to a string. The following patch works around this: Index: console.py =================================================================== — console.py (revision 1372526) +++ console.py (working copy) @@ -2443,6 +2443,8 @@ uid = self.conn.user_id if uid._ class _ == tuple and len(uid) == 2: self.saslUser = uid [1] + elif type(uid) is str: + self.saslUser = uid; else: self.saslUser = None

          People

          • Assignee:
            kgiusti Ken Giusti
            Reporter:
            kgiusti Ken Giusti
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development