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.

Sign up for GitHub

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

Jump to bottom

stable enough to use? #2

Closed
gurachan opened this issue Nov 22, 2019 · 5 comments
Closed

stable enough to use? #2

gurachan opened this issue Nov 22, 2019 · 5 comments
Labels
question Further information is requested

Comments

@gurachan
Copy link

No description provided.

@gurachan
Copy link
Author

aw it too slow I don't know what happens but the loading time now is bad :( like 10 seconds difference

@gurachan
Copy link
Author

from real-time to 10 seconds delay

@mmdevries
Copy link
Owner

To be honest, I have no idea what you mean.
Can you be more specific on your issue?

@gurachan
Copy link
Author

I use it on socket.io lately and it became slow. like 10 sec to do back and forth

@mmdevries
Copy link
Owner

Your question is if this module is stable enough. Well we use it for weeks already on our production platforms 24/7 without issues.

Do you now have suddenly an issue with this module? If so, which version of socket.io and uws are you using now? And on which version of socket.io and uws it was still fast?

@mmdevries mmdevries added the question Further information is requested label Nov 27, 2019
darrachequesne added a commit to socketio/engine.io that referenced this issue Feb 6, 2020
@darrachequesne
test: add Node.js 10, 12 and 13 in the test matrix
61e639b 
uws is no longer maintained, and did not support Node.js >= 10.

We now use a fork in order to support the newer versions of Node.js.
It is maintained there: https://github.com/mmdevries/uws

And can be installed with: `npm i github:mmdevries/uws#2.4.1`

We could also try to support uWebSockets.js, but it does not have the
same API as ws.

The "engines" attribute has also been added in the package.json file,
since we broke the support for Node.js 6 in the latest minor release.

Source: https://github.com/mmdevries/uws
Related: #583
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gurachan @mmdevries