Qpid
  1. Qpid
  2. QPID-3733

QpidRAManagedConnection setup() method is incorrect for non-XA transaction state as well as username/password

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.15
    • Component/s: JCA
    • Labels:
      None
    • Environment:

      All OS platforms, all JEE supported platforms

      Description

      The QpidRAManagedConnection class does not correctly handle non-XA transaction state as well as username and password information in the call to creating the underlying connection. For the non-XA transaction state, we don't account for the user specifying transacted=true for the Session. The username password info is completely ignored for non-XA connections.

      1. QPID-3733.patch
        11 kB
        Weston M. Price
      2. QPID-3733.patch
        5 kB
        Weston M. Price

        Activity

        Weston M. Price created issue -
        Hide
        Weston M. Price added a comment -

        Fixes issues with both non-XA connections as well as setting the username/password correctly on connection creation.

        Show
        Weston M. Price added a comment - Fixes issues with both non-XA connections as well as setting the username/password correctly on connection creation.
        Weston M. Price made changes -
        Field Original Value New Value
        Attachment QPID-3733.patch [ 12509753 ]
        Weston M. Price made changes -
        Assignee Weston M. Price [ wprice ]
        Hide
        Weston M. Price added a comment -

        Tested with both JBoss EAP 5.x and Geronimo 2.2.x

        Show
        Weston M. Price added a comment - Tested with both JBoss EAP 5.x and Geronimo 2.2.x
        Hide
        Weston M. Price added a comment -

        Fixed with attached patch. Tested in EAP/Geronimo environments.

        Show
        Weston M. Price added a comment - Fixed with attached patch. Tested in EAP/Geronimo environments.
        Weston M. Price made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Weston M. Price added a comment -

        Issues have come up with our internal TCK that require the re-opening of this issue.

        Show
        Weston M. Price added a comment - Issues have come up with our internal TCK that require the re-opening of this issue.
        Weston M. Price made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Hide
        Weston M. Price added a comment -

        Latest patch for QPID-3733.

        Show
        Weston M. Price added a comment - Latest patch for QPID-3733 .
        Weston M. Price made changes -
        Attachment QPID-3733.patch [ 12512761 ]
        Hide
        Weston M. Price added a comment -

        Fixed in trunk with provided patch.

        Show
        Weston M. Price added a comment - Fixed in trunk with provided patch.
        Weston M. Price made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        4d 1h 38m 1 Weston M. Price 10/Jan/12 21:27
        Resolved Resolved Reopened Reopened
        5d 18h 17m 1 Weston M. Price 16/Jan/12 15:45
        Reopened Reopened Resolved Resolved
        54d 20m 1 Weston M. Price 10/Mar/12 16:06

          People

          • Assignee:
            Weston M. Price
            Reporter:
            Weston M. Price
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development