-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#403 Provide SockJS config for minimum transport fallback timeout #413
Conversation
sockjs#403 Provide SockJS config for minimum transport fallback timeout
Have the same problem with a short timeout. |
After using this fix, the customers behind The Great Firewall have no more complaints |
+1. Please merge 👍 |
This is very much needed. 👍 |
+1 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not make changes to dist
or lib/version.js
in your PRs.
mark |
Have the same problem. Please, add this functionality. |
This is a small fix for a major issue. |
Fixed in 537352c |
One of our clients experiences a poor network QoS and falling back to xhr makes it even worse. A simple minimum timeout could solve the issue #403