Closed Jared-Sprague closed 8 years ago
Here are the messages. Client on:
Client.emit:
Server.on:
Pre-requisite #245
Just found this tool called artillery. Maybe you found it before but just in case:
https://github.com/shoreditch-ops/artillery
It's a node-based load tester for http and ws. We could add it as a dev dependency and add a npm run load
to test load stats every time we make a server change! Cool to think about.
That sounds cool!
@mwcz so after your review, is this ready to merge?
@Jared-Sprague yep! Everything looked great to me and I couldn't find any bugs.
@mwcz Thanks for the review! Merged and closed!
After doing performance tests it's clear that WebSocket has a performance advantage over Socket.io, see #157.
Convert Network.js and server.js to use WebSocket