Flood protection needs revision
Status: Beta
Brought to you by:
ziggythehamster
Our flood protection is pretty.. well.. simple. If a
line is greater than a certain length, the user is
disconnected. Here are some things that need to be changed:
- The error message "Max temp. SendQ exceeded" needs to
be changed to either "Flooding", or send an error back
to the client along the lines of "Sorry, the line you
sent was too long.". The latter option is the best
option, but it poses a slight problem: flooders can do
double the damage by flooding. So I need to make it
either ignore the user for a few seconds (best choice),
or warn + kill.
- Be more forgiving with line endings. Some clients use
LF, others use CRLF. And some use CR. Need to make
everything LF and parse it like that or something.