Skip to content

Compact binary SignalR protocol without MessagePack dependency #7387

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

Closed
danroth27 opened this issue Feb 8, 2019 · 4 comments
Closed

Compact binary SignalR protocol without MessagePack dependency #7387

danroth27 opened this issue Feb 8, 2019 · 4 comments
Labels
area-signalr Includes: SignalR clients and servers ✔️ Resolution: Duplicate Resolved as a duplicate of another issue
Milestone

Comments

@danroth27
Copy link
Member

For Razor Components we use a SignalR hub to manage the UI interactions. We need these interactions to be as low latency as possible. Currently we use MessagePack, but we need a default protocol implementation that doesn't have a third party dependency so that it can be fully serviced.

@davidfowl @bradygaster @SteveSandersonMS @mkArtakMSFT

@danroth27 danroth27 added the area-signalr Includes: SignalR clients and servers label Feb 8, 2019
@AArnott
Copy link
Contributor

AArnott commented Feb 9, 2019

Would a fork of MessagePack satisfy? The one I'm working on is based on neuecc's (and we're collaborating to stay close at this point). I'm currently refactoring and tuning it to work with all the latest .NET Core APIs (e.g. IBufferWriter<byte> and ReadOnlySequence<byte>).

@rynowak
Copy link
Member

rynowak commented Feb 11, 2019

@AArnott - whatever we ship for Razor Components needs to be in the box and have the same support policy as the rest of ASP.NET Core. We're planning to do a reader/writer for this since that meets our requirements to use it.

@davidfowl
Copy link
Member

After spending some time helping @AArnott refactor MessagePackCSharp, it might make sense to grab and MessagePackWriter and MessagePackReader he's currently writing and bake this into RazorComponents. It's basically a better version of @pranavkm 's prototype 😄

@pranavkm
Copy link
Contributor

Closing in favor of #7311 which has more details.

@pranavkm pranavkm added the ✔️ Resolution: Duplicate Resolved as a duplicate of another issue label Mar 29, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-signalr Includes: SignalR clients and servers ✔️ Resolution: Duplicate Resolved as a duplicate of another issue
Projects
None yet
Development

No branches or pull requests

6 participants