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

Deprecate the socket.io library in favor of move to native client-side WebSocket interface. #4314

Open
aaclayton opened this issue Dec 16, 2020 · 3 comments
Labels
api Issues related to the API used by Mod Devs tech-debt Issues focused on the reduction of technical debt

Comments

@aaclayton
Copy link
Contributor

aaclayton commented Dec 16, 2020

We may be able to eliminate the client-side socket.io client in favor of native WebSocket integration.

This would require implementing a few bespoke features where we currently rely on Socket.io: https://socket.io/docs/v4/#features

@aaclayton aaclayton removed this from the Epics milestone Jun 11, 2022
@aaclayton aaclayton changed the title Evaluate deprecation of socket.io library in favor of move to native client-side WebSocket interface. Deprecate the socket.io library in favor of move to native client-side WebSocket interface. Nov 6, 2022
@aaclayton aaclayton added api Issues related to the API used by Mod Devs and removed epic Major issues which represent substantial chunks of work labels Jul 17, 2023
@CodaBool
Copy link

I'm sure you're pretty versed on websockets but thought I'd post some relevant info. While websockets are commonly native now they would still lack two important features socket.io offers. These would need to be implemented personally:

  • Auto reconnects (using heartbeats)
  • packet buffering (packets are resent on re-connection).

They have some more detail on their landing page.

@aaclayton
Copy link
Contributor Author

Yes, absolutely right. These features (and also especially acknowledgements) are key to our use case. It's possible we will deprioritize this (again) as we have for previous versions because making this change is not an urgent need and we have plenty of other things that require doing in V12.

@aaclayton
Copy link
Contributor Author

Deprioritizing from V12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Issues related to the API used by Mod Devs tech-debt Issues focused on the reduction of technical debt
Projects
None yet
Development

No branches or pull requests

2 participants