search

timotheeg / nestrischamps

A web-based OCR and restreamer system for NES Classic Tetris players
MIT License
48 stars 12 forks source link

Replace websocket by uWebsocket #45

Closed timotheeg closed 1 week ago

timotheeg commented 3 years ago

uWebSockets seems to be the most performant websocket library for nodejs.

TODO: replace ws by uws

Looks like uWebSocket cannot be installed with npm, so it might mean migrate the project to using yarn instead, which is fine I guess?

timotheeg commented 1 week ago

Won't do. uWebSockets requires taking over both express and ws, which is not worth it.

I'd reopen only if performance becomes an obvious bottleneck on the server.