[AMQ-3953] org.apache.activemq.util.URISupport.isCompositeURI doesn't work properly.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.6.0
• Fix Version/s: 5.7.0
• Component/s: Broker
• Patch Info: Patch Available
• Assignee: Timothy A. Bish
• Reporter: Mikhail Melamud
• Created: 30/Jul/12 22:44
• Updated: 31/Jul/12 15:45
• Resolved: 31/Jul/12 15:45

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...