]> git.openstreetmap.org Git - nominatim.git/commitdiff
disable ua direct blocks
authorSarah Hoffmann <lonvia@denofr.de>
Thu, 28 May 2015 19:28:44 +0000 (21:28 +0200)
committerSarah Hoffmann <lonvia@denofr.de>
Thu, 28 May 2015 19:28:44 +0000 (21:28 +0200)
utils/cron_ipanalyse.py

index 333dbed55b4409da28081d24a92c22debad49e1f..05b0b7f9e18382e05c33ba6a0c67d69bbc8050d0 100755 (executable)
@@ -241,12 +241,12 @@ class IPstats:
         if was_blocked:
             # deblock only if the IP has been really quiet
             # (properly catches the ones that simply ignore the HTTP error)
-            return None if self.long_total < 5 else 'block'
+            return None if self.long_total < 20 else 'block'
         if self.long_api > BLOCK_UPPER or self.short_api > BLOCK_UPPER / 3:
                 # client totally overdoing it
                 return 'block'
         if was_bulked:
-            if self.short_total < 5:
+            if self.short_total < 20:
                 # client has stopped, debulk
                 return None
             if self.long_api > BLOCK_LIMIT or self.short_api > BLOCK_LIMIT / 3:
@@ -255,8 +255,8 @@ class IPstats:
             return 'bulk'
 
         if self.long_api > BULKLONG_LIMIT or self.short_api > BULKSHORT_LIMIT:
-            if self.bad_ua:
-                return 'uablock' # bad useragent
+            #if self.bad_ua:
+            #    return 'uablock' # bad useragent
             return 'bulk'
 
         return None
@@ -317,7 +317,8 @@ if __name__ == '__main__':
     # if the bulk pool is still empty, clients will be faster, avoid having
     # them blocked in this case
     if numbulks < 10:
-        BLOCK_LIMIT = 2*BLOCK_UPPER
+        BLOCK_UPPER *= 2
+        BLOCK_LIMIT = BLOCK_UPPER
 
 
     # collecting statistics