search

Matthias247 / jawampa

Web Application Messaging Protocol (WAMP v2) support for Java
Apache License 2.0
148 stars 56 forks source link

Fix for issue#71 #72

Closed bhuvanalakshmi closed 8 years ago

bhuvanalakshmi commented 8 years ago

Changed makeSubscription method to return Observable of type MessageInfo which contains topic and message. This will give access to the actual topic name.

Matthias247 commented 8 years ago

Thanks for the contribution! I think adding some code that helps for your use case with wildcard subscriptions is valid concern. I added some comments regarding your implementation. Could you try to address these?

bhuvanalakshmi commented 8 years ago

I have addressed your comments, please review and let me know

Matthias247 commented 8 years ago

look good. thanks