socket-io-game icon indicating copy to clipboard operation
socket-io-game copied to clipboard

Spamming io(); is a potential DoS on the server.

Open dunnousername opened this issue 7 years ago • 1 comments

Hello, I'm not good with programming much and am mostly self taught; however, I noticed that creating many connections per IP is possible simply by executing io(); or a while(condition) io();. I'm not completely sure how socket.io works, but this is a potential security flaw.

Perhaps limiting the connections per IP is an option?

dunnousername avatar Sep 24 '17 03:09 dunnousername

A possible fix might be, after created the initial socket via io(), setting io = undefined. It seems a bit hacky, but it should work.

zac-garby avatar Sep 24 '17 17:09 zac-garby