Skip to content
This repository has been archived by the owner on Nov 20, 2018. It is now read-only.

WebSockets does not have support for extensions #711

Closed
vanillajonathan opened this issue Sep 19, 2016 · 5 comments
Closed

WebSockets does not have support for extensions #711

vanillajonathan opened this issue Sep 19, 2016 · 5 comments

Comments

@vanillajonathan
Copy link
Contributor

The WebSocket class does not have support for extensions. Which is part of the rfc6455 standard:
https://tools.ietf.org/html/rfc6455#section-9

It should read the Sec-WebSocket-Extensions HTTP header from the client.

The WebSocket class does not expose a Extensions property.

@vanillajonathan
Copy link
Contributor Author

Depends on upstream support in System.Net.WebSockets.

Depends on issue dotnet/corefx#11874.

@Tratcher
Copy link
Member

Did you have any specific extensions you want to use?

@vanillajonathan
Copy link
Contributor Author

No, this exposes to the server the extension(s) that the client requests.
Then the server have to negotiate with the client and send back which extensions it supports.

Then any extension has to be implemented by the client and the server on top of this which only exposes the requested extension.

@Tratcher
Copy link
Member

Note this hasn't been done yet because there has been little interest expressed for any websocket extensions. We're unlikely to add the the property if we don't think it will actually get used.

@muratg muratg changed the title WebSocket does not have support for extensions WebSockets does not have support for extensions Sep 23, 2016
@muratg muratg added this to the Backlog milestone Sep 23, 2016
@aspnet-hello
Copy link

This issue was moved to dotnet/aspnetcore#2715

@aspnet aspnet locked and limited conversation to collaborators Jan 2, 2018
@aspnet-hello aspnet-hello removed this from the Backlog milestone Jan 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants