aio-libs / sockjs

SockJS Server

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

RawWebSocketTransport reason?

pahaz opened this issue · comments

Hi! I check the sockjs code and look at RawWebSocketTransport.

Confusion:

We have Session abstraction and _remote_message(msg) coro link which calling on new message from client.

Take a look at websocket transport link
and rawwebsocket transport link
.

As we can see in the first case we pass the dict object and in the secon case we pass the str.

Next. as we can see here the RawWebSocketTransport is not a part of the sockjs transports.
And uses as additional route here

I thik that the RawWebSocketTransport used for tests or something similar. And this is not a part of the sockjs lib.

What is the real purpose of the RawWebSocketTransport?

interesting remark: _remote_message(msg) require str object (because msg[:200]). But transports pass the json loaded object (bay be dict).

Is this bug or feature?