Handle info-style messages to the session

sockjs / sockjs-erlang

WebSocket emulation - Erlang server

http://sockjs.org

Other

266 stars 120 forks source link

Handle info-style messages to the session #29

Closed puzza007 closed 12 years ago

majek commented 12 years ago

So the idea is to be able to capture messages on the session process. I always assumed that the proper way is to spawn a new process (gen server) and do this stuff there.

But I'm fine with the patch.

Few questions:

do we need to update README?
is it compatible with the current code? (ie: what happens if user doesn't define sockjs_info?)

puzza007 commented 12 years ago

Excellent

I've updated the example in the README
It should work the same as existing code when sockjs_info isn't defined (i.e. crash). Plus since sockjs_info is specified in the behaviour code that doesn't implement it will emit a warning.

majek commented 12 years ago

Great. Applied to dev branch: https://github.com/sockjs/sockjs-erlang/commit/fae513e155785e9861462ab6f92ec3aacc34e8f7