Qpid
  1. Qpid
  2. QPID-3373

Java client: connectionFactory.createConnection method should not rewrite stored credentials

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.10
    • Fix Version/s: 0.14
    • Component/s: Java Client
    • Labels:
      None

      Description

      Method connectionFactory.createConnection("foo", "bar") causes than Java client caches the credentials and overwrites default ones provided in connection URL. Thus subsequent calling of connectionFactory.createConnection() method (without a parameter) results in using the credentials "foo" / "bar" despite default ones should be used.

      How reproducible:
      100%

      Steps to Reproduce:
      1. Setup a broker with PLAIN authentication
      2. Compile attached JUnit test
      3. Run it (java org.junit.runner.JUnitCore authNoCredentialsInSecondConnection)

      Actual results:
      In try .. catch block, connectionFactory2.createConnection(); does not raise an exception that authentication failed. (and thus fail command below raises java.lang.AssertionError exception). In fact, connectionFactory2.createConnection(); uses "guest" / "guest" credentials set for connectionFactory1.

      Expected results:
      Authentication fails for connectionFactory2.createConnection(); as it uses default credentials from URL (what is "dummy" / "dummy").

        Activity

        Pavel Moravec created issue -
        Pavel Moravec made changes -
        Field Original Value New Value
        Attachment qpid-java-0.10.connection_credentials.patch [ 12487968 ]
        Pavel Moravec made changes -
        Pavel Moravec made changes -
        Attachment qpid-java-0.10.connection_credentials.patch [ 12487968 ]
        Pavel Moravec made changes -
        Comment [ patch proposal ]
        Pavel Moravec made changes -
        Rajith Attapattu made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Rajith Attapattu [ rajith ]
        Fix Version/s 0.14 [ 12316855 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Rajith Attapattu
            Reporter:
            Pavel Moravec
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2h
              2h
              Remaining:
              Remaining Estimate - 2h
              2h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development