activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Timothy Bish (JIRA)" <j...@apache.org>
Subject [jira] Commented: (AMQCPP-352) FailoverTransportFactory doesn't work correctly with more than two composite uris
Date Sun, 27 Feb 2011 19:11:36 GMT

    [ https://issues.apache.org/jira/browse/AMQCPP-352?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13000028#comment-13000028
] 

Timothy Bish commented on AMQCPP-352:
-------------------------------------

Would be helpful if you could include a test case in URISupportTest to demonstrate your issue,
this will ensure that the issue gets fixed and remains fixed.

> FailoverTransportFactory doesn't work correctly with more than two composite uris 
> ----------------------------------------------------------------------------------
>
>                 Key: AMQCPP-352
>                 URL: https://issues.apache.org/jira/browse/AMQCPP-352
>             Project: ActiveMQ C++ Client
>          Issue Type: Bug
>          Components: Transports
>    Affects Versions: 3.0, 3.2.4
>         Environment: All platforms
>            Reporter: Dmitry Babich
>            Assignee: Timothy Bish
>
> Only first and last composite uris are recognized by FailoverTransportFactory.
> For example - given uri: 
>    failover://(tcp://10.10.10.1:61616,tcp://10.10.10.2:61616,tcp://10.10.10.3:61616,tcp://10.10.10.4:61616)
> only "tcp://10.10.10.1:61616" and "tcp://10.10.10.4:61616" will work.
> The bug in URISupport::splitComponents(). 
> Line 350:
>       std::string s = str.substr( last, i );
> should be replaced by:
>       std::string s = str.substr( last, i - last );  // 2nd argument of substr() is length
of substring

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message