You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If the websocket is sending a large message over a slow link to the client, then it may take more than 10 seconds for the heartbeat to actually arrive. A big improvement would be to wait until the heartbeat is actually sent to start the timer, but even then, the client may have large messages it's sending to the server too, which a heartbeat response would be queued behind.
I'm not necessarily asking for a change to the default behavior, but it would be great to be able to disable or configure this timeout, like disconnect_delay for the xhr-based transports.
(If you like I can put together a PR for this, but I wanted to first ask if such a patch would be welcome.)
The text was updated successfully, but these errors were encountered:
The websocket transport has a hardcoded 10 second timer:
sockjs-node/src/trans-websocket.coffee
Line 99 in 61bba46
If the websocket is sending a large message over a slow link to the client, then it may take more than 10 seconds for the heartbeat to actually arrive. A big improvement would be to wait until the heartbeat is actually sent to start the timer, but even then, the client may have large messages it's sending to the server too, which a heartbeat response would be queued behind.
I'm not necessarily asking for a change to the default behavior, but it would be great to be able to disable or configure this timeout, like
disconnect_delay
for the xhr-based transports.(If you like I can put together a PR for this, but I wanted to first ask if such a patch would be welcome.)
The text was updated successfully, but these errors were encountered: