Closed essen closed 9 years ago
:+1: to /ws
.
Tasks left to do:
After looking in more details, rabbit_ws_sockjs is actually little about SockJS. It's basically starting the Cowboy listeners (including /ws) and then there's about 10 lines that are SockJS-specific. If it was to be split, I would prefer to do so in a different larger refactoring PR.
@essen gotcha, lets tick that one off then and worry about it later (if at all).
I think we're good now.
We want to add a new path, for example /ws, that would serve STOMP over raw Websocket, ie without the SockJS dependency.
Initially and by default it would serve STOMP, and later on it may be expanded to use different protocols (using Websocket's protocol header). There's however no need to take care of this until we have a second protocol to use.
The rabbitmq-web-stomp-examples plugin also needs to be updated and provide an alternative for each example for raw Websocket.
For the sake of simplicity, and to avoid duplicating code, I propose to do this in two steps:
This ticket is about the first step.