activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hiram Chirino (JIRA)" <j...@apache.org>
Subject [jira] [Issue Comment Edited] (APLO-6) Alternative topic separators should be supported in Apollo, as in ActiveMQ
Date Wed, 18 May 2011 21:58:47 GMT

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

Hiram Chirino edited comment on APLO-6 at 5/18/11 9:56 PM:
-----------------------------------------------------------

Awesome thanks.  Ok simplest thing would be to make this an additional stomp protocol option
as documented at:

http://activemq.apache.org/apollo/documentation/user-manual.html#Stomp_Protocol_Options

something like:

<connector id="tcp" bind="tcp://0.0.0.0:61613">
  <stomp path_separator="/"/>
</connector>


      was (Author: chirino):
    Awesome thanks.  Ok simplest thing would be to make this an additional stomp protocol
option as documented at:

http://activemq.apache.org/apollo/documentation/user-manual.html#Stomp_Protocol_Options]

something like:

<connector id="tcp" bind="tcp://0.0.0.0:61613">
  <stomp path_separator="/"/>
</connector>

  
> Alternative topic separators should be supported in Apollo, as in ActiveMQ
> --------------------------------------------------------------------------
>
>                 Key: APLO-6
>                 URL: https://issues.apache.org/jira/browse/APLO-6
>             Project: ActiveMQ Apollo
>          Issue Type: New Feature
>          Components: apollo-broker
>    Affects Versions: 1.0-beta2
>         Environment: All environments.
>            Reporter: Sam Starling
>            Assignee: Hiram Chirino
>            Priority: Minor
>              Labels: apollo, topic
>
> When using a character other than a period as a topic separator (.), Apollo throws the
following exception:
> {code} 
> org.apache.activemq.apollo.util.path.PathParser$PathException: Invalid destination path
part: 'event/abc', it does not match regex: [a-zA-Z0-9\_\-\%\~]+
> {code}
> In the example, we're using {{/topic/event/abc}} as a topic. In {{[PathParser.java|http://svn.apache.org/viewvc/activemq/activemq-apollo/trunk/apollo-util/src/main/scala/org/apache/activemq/apollo/util/path/PathParser.java?view=markup]}},
line 47 determines that {{path_seperator = new AsciiBuffer(".")}}. Thus, Apollo incorrectly
tokenizes the topic and so {{event/abc}} fails the regular expression on line 48 of the same
file.
> In short, it should be possible to specify what the topic separator is, or (even better)
detect when a different character is being used as a delimiter.

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

Mime
View raw message