activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dmitry Babich (JIRA)" <j...@apache.org>
Subject [jira] Updated: (AMQCPP-352) FailoverTransportFactory doesn't work correct with more than 2 composite uris
Date Sun, 27 Feb 2011 18:33:37 GMT

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

Dmitry Babich updated AMQCPP-352:
---------------------------------

    Description: 
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() in line 350. 
Line:
   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


  was:
Only first and last composite uris are recognized by FailoverTransportFactory.

For example - given 
   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() in line 350. Line:
   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



> FailoverTransportFactory doesn't work correct with more than 2 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
>         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() in line 350. 
> Line:
>    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