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

Switch from Web Sockets to Long polling #437

Open
virgile-dev opened this issue Nov 20, 2024 · 1 comment · May be fixed by #517
Open

Switch from Web Sockets to Long polling #437

virgile-dev opened this issue Nov 20, 2024 · 1 comment · May be fixed by #517
Assignees

Comments

@virgile-dev
Copy link
Collaborator

Bug Report

Problematic behavior
Websockets are not compatible with internal networks and security configuration.

Expected behavior/code
Replace websocket with Long Polling.
Which means we need to make changes to hoccus poccus and Y.js

@YousefED
Copy link
Collaborator

Thanks for adding this. Did we confirm this is really the issue for certain users?

Kevin's first thoughts on this are here: ueberdosis/hocuspocus#95 (comment) and this specifically might be concerning:

A quick web search revealed that the very same Antivirus software that can do this if (mis)configured, also blocks long-pulling

I'm quite sure we can add support for it but it will be quite a bit of work - so it would be great if we can test whether it actually fixes the issue for those setups

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Doing next
Development

Successfully merging a pull request may close this issue.

3 participants