PowerDNS / weakforced

Anti-Abuse for servers at authentication time
GNU General Public License v3.0
125 stars 33 forks source link

Don't record stats or log for requests which are actually just EOF #311

Closed neilcook closed 4 years ago

neilcook commented 4 years ago

It's probably worth explaining the rationale behind this. I noticed when testing a docker container with wforce that the stats were getting double-counted. I realised that POLLIN can be present just to indicate EOF. Adding an explicit EOF check, and only incrementing stats if !EOF fixes the double-counting issue.