[jira] Created: (AMQ-671) CompositeTransportChannelFactory doesn't handle some connection URLs with list configuration options

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (AMQ-671) CompositeTransportChannelFactory doesn't handle some connection URLs with list configuration options

JIRA jira@apache.org
CompositeTransportChannelFactory doesn't handle some connection URLs with list configuration options
-----------------------------------------------------------------------------------------------------

         Key: AMQ-671
         URL: https://issues.apache.org/activemq/browse/AMQ-671
     Project: ActiveMQ
        Type: Bug

  Components: Transport  
    Versions: 3.2.2    
    Reporter: Ted X. Toth


A connection URL with list configuration options (maximumRetries, failureSleepTime, etc ...) which doesn't put it parens around it's connection string as in '(tcp://localhost:61616)' is not processed properly because URI.getQuery method return null instead of the configuration options.

CompositeTransportChannelFactory.java:            answer = uri.getQuery();

For example:
reliable:(tcp://foo:61699,tcp://bar:61617,tcp://whatnot:61698)?maximumRetries=5
will work but:
reliable:tcp://foo:61699?maximumRetries=5
will not work.

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   https://issues.apache.org/activemq/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply | Threaded
Open this post in threaded view
|

[jira] Resolved: (AMQ-671) CompositeTransportChannelFactory doesn't handle some connection URLs with list configuration options

JIRA jira@apache.org
     [ https://issues.apache.org/activemq/browse/AMQ-671?page=all ]
     
james strachan resolved AMQ-671:
--------------------------------

    Resolution: Won't Fix

This was done by design.

The reason is, any transport at any point inside the URI can have properties which are configured via the URI query notation. So we use () as a way of specifying to which URI the parameters refer to

> CompositeTransportChannelFactory doesn't handle some connection URLs with list configuration options
> ----------------------------------------------------------------------------------------------------
>
>          Key: AMQ-671
>          URL: https://issues.apache.org/activemq/browse/AMQ-671
>      Project: ActiveMQ
>         Type: Bug

>   Components: Transport
>     Versions: 3.2.2
>     Reporter: Ted X. Toth

>
>
> A connection URL with list configuration options (maximumRetries, failureSleepTime, etc ...) which doesn't put it parens around it's connection string as in '(tcp://localhost:61616)' is not processed properly because URI.getQuery method return null instead of the configuration options.
> CompositeTransportChannelFactory.java:            answer = uri.getQuery();
> For example:
> reliable:(tcp://foo:61699,tcp://bar:61617,tcp://whatnot:61698)?maximumRetries=5
> will work but:
> reliable:tcp://foo:61699?maximumRetries=5
> will not work.

--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   https://issues.apache.org/activemq/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira