[ 
https://issues.apache.org/jira/browse/AMQ-3953?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Mikhail Melamud updated AMQ-3953:
---------------------------------

    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:{color:blue} 
_failover://(tcp://bla:61616,tcp://bla:61617)}_{color} 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 {quote}uri 
is composite if the scheme is followed by (..){quote}

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

  was: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:{color:blue} 
_failover://(tcp://bla:61616,tcp://bla:61617)}_{color} 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 {quote}uri 
is composite if the scheme is followed by (..){quote}

    
> org.apache.activemq.util.URISupport.isCompositeURI doesn't work properly.
> -------------------------------------------------------------------------
>
>                 Key: AMQ-3953
>                 URL: https://issues.apache.org/jira/browse/AMQ-3953
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 5.6.0
>            Reporter: Mikhail Melamud
>         Attachments: URISupport-patch.txt, URISupportTest-patch.txt
>
>
> 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:{color:blue} 
> _failover://(tcp://bla:61616,tcp://bla:61617)}_{color} 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 {quote}uri 
> is composite if the scheme is followed by (..){quote}
> Please, find attached unit test patch and, possible fix based on assumption 
> that parentheses is a criteria...  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to