Change rate limiting to be per-UUID
RalfJung opened this issue · comments
@RobWei recently contributed per-IP rate limiting. I was excited to try and see if this helps mitigate #143, but unfortunately that is not the case: the bad node reconnects with a rate that is too low for reasonable rate limiting, in particular when considering that people could run multiple nodes under the same IP.
Hence I propose to change that rate-limiting to be per-UUID instead of per-IP. The motivating log in #138 shows the same UUID over and over, so this approach should still help @RobWei, but at the same time it means we can set the rate limit much lower because we do not have to worry about the same IP being used by multiple clients.