forked from sockjs/sockjs-ruby
-
Notifications
You must be signed in to change notification settings - Fork 5
Expand file tree
/
Copy pathTODO
More file actions
24 lines (13 loc) · 602 Bytes
/
Copy pathTODO
File metadata and controls
24 lines (13 loc) · 602 Bytes
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
Catch and display uncaught exceptions - tricky because of async
Rack::SockJS should expect to be mapped, not get called with a prefix
Extract FSM execution to module
Unpack Transport#response - it's three methods
Refactor graph of Transport/Response/Session/Buffer - it's crazytown
Study and document relationship of actors in the responses
Why is Rack::SockJS difference from SockJS::Connection? Merge or document
Override pattern for get_session (or at least find_session)
format_frame/1 -> format_frame/2
open!/0 -> open!/2
Remove alter_ego
Review callbacks...
Review/remove Forwardable