AndrewBelt / hack.chat

a minimal, distraction-free chat application
https://hack.chat/
MIT License
2.39k stars 337 forks source link

Added forced flushing on socket overflow #132

Closed marzavec closed 7 years ago

marzavec commented 7 years ago

The previous patch was working for browser-based attacks, however scripted software was still able to cause an overflow.

The socket.close() function did not seem to act quickly enough in this case. I was wrong earlier about not being able to flush the socket buffer & have added that into this patch. In local tests on 2 different machines (using a software-based attack) this patch is tested and working (again).

AndrewBelt commented 7 years ago

Thanks!