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
Append "/poll" to polling attempts in case the URL ends with "/hubs" #1767
Conversation
- Append "/poll" to polling attempts in case the URL ends with "/hubs"
- Append "/poll" to polling attempts in case the URL ends with "/hubs"
@@ -355,7 +355,7 @@ | |||
if (index >= transports.length) { | |||
if (!connection.transport) { | |||
// No transport initialized successfully | |||
$(connection).triggerHandler(events.onError, "SignalR: No transport could be initialized successfully. Try specifying a different transport or none at all for auto initialization."); | |||
$(connection).triggerHandler(events.onError, ["SignalR: No transport could be initialized successfully. Try specifying a different transport or none at all for auto initialization."]); |
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.
?
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.
triggerHandler expects an array.
It came up when I was trying to figure out why my JS test was failing in Chutzpah but not in normal browsers.
I made this change in its own commit.
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.
Ok
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.
Are we missing this in other cases?
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.
Nope
Make sure you run some tests with old clients and new servers and let @Xiaohongt know as well. |
@davidfowl There are absolutely no server-side changes making it impossible for this change to affect old clients. |
@halter73 I still want verification |
#1707