Skip to content
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

2017 - Support multiple protocols server side #2029

Closed
wants to merge 4 commits into from
Closed

Conversation

NTaylorMullen
Copy link
Contributor

The server will now push down protocols based on a client given protocol in negotiate. There is a min and a max protocol version and the server will only server >= min or <= max protocol versions
#2017

url = signalR.transports._logic.addQs(url, connection.qs);

// Add the client version to the negotiate request. We utilize the same addQs method here
// so that it can append the clientVersion the appropriately way to the URL
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fail english

- Supports ranges for server protocols
- Also allows replacement of protocol resolver

#2017
- Also added the feature to pass the clientProtocol in the negotiation request

#2017
- Also added the feature to pass the clientProtocol in the negotiation request

#2017
- Includes JS, C# Client, and Server
- Had to add a little bit of extensibility to the test utilities for JS to allow overriding of the WrapConnectionStart functionality

#2017
@NTaylorMullen NTaylorMullen deleted the 2017 branch May 16, 2013 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants