[jira] [Commented] (CAMEL-10949) Websocket clients get message from all resources on one port

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on CAMEL-10949:
----------------------------------------

GitHub user onders86 opened a pull request:

    https://github.com/apache/camel/pull/1590

    CAMEL-10949 - websocket clients will get messages on which uri they a…

    …re subscribed to if sendToAll set to true

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/onders86/camel CAMEL-10949

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/1590.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1590
    
----
commit b4c9ec2db90e4dc4d8e9e20dbe85be748976237e
Author: onders86 <on...@gmail.com>
Date:   2017-04-03T13:49:31Z

    CAMEL-10949 - websocket clients will get messages on which uri they are subscribed toif sendToAll set to true

----


> Websocket clients get message from all resources on one port 
> -------------------------------------------------------------
>
>                 Key: CAMEL-10949
>                 URL: https://issues.apache.org/jira/browse/CAMEL-10949
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-websocket
>            Reporter: Ravi Godbole
>            Assignee: onder sezgin
>            Priority: Minor
>
> Routebuider configuration in which only resource URI is different while port is same.
> 1. newsTopic
> 2. stockQuoteTopic
>  Ideally other clients connecting to newsTopic should get messages published
>  on this topic only but they are also getting messages published on stockQuoteTopic.
>  On further investigation I found out that memory store used to by sendAll
>  is shared by all resource uri for a given port in the server.
>  from("activemq:topic:newsTopic").to("websocket://
>  0.0.0.0:9090/newsTopic?sendToAll=true");
>         from("activemq:topic:stockQuoteTopic").to("websocket://
>  0.0.0.0:9090/stockQuoteTopic?sendToAll=true");
>  Sample Ouput received from client 
>  2017-03-06 15:35:15.789:INFO::main: Logging initialized @313ms
>  Connected to server
>  Message received from server:MSFT        [ published on stockQuoteTopic. Expected ..]
>  Message received from server:CNBC News   [ published on newsTopic. should not have received ]
>  Message received from server:BBC News    [ published on newsTopic. should not have received ]



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)