Uploaded image for project: 'ActiveMQ'
  1. ActiveMQ
  2. AMQ-3953

org.apache.activemq.util.URISupport.isCompositeURI doesn't work properly.

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.6.0
    • Fix Version/s: 5.7.0
    • Component/s: Broker
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      URISupport.isCompositeURI(URI uri) is not implemented correctly. It doesn't work for the cases when a scheme part contains "//" (double slashes). In the simple test where a failover uri contains two url's: failover://(tcp://bla:61616,tcp://bla:61617)} and is the composite URI, URISupport.isCompositeURI(uri) returns false... The general problem is that the definition of a "composite URI" is pretty loose and is not defined formally... According to http://activemq.2283324.n4.nabble.com/composite-uri-td4654365.html

      uri is composite if the scheme is followed by (..)

      Please, find attached unit test patch and, possible fix based on assumption that parentheses is a criteria...

        Attachments

        1. URISupport-patch.txt
          0.8 kB
          Mikhail Melamud
        2. URISupportTest-patch.txt
          3 kB
          Mikhail Melamud

          Activity

            People

            • Assignee:
              tabish Timothy A. Bish
              Reporter:
              mikmela Mikhail Melamud
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: