reduce cache durations

Author: mobula9

docs/complete-guide.md

@@ -77,7 +77,8 @@ echo "BOUNCER_KEY=`docker-compose exec crowdsec /usr/local/bin/cscli bouncers ad
 Init the `composer.json` and download the `crowdsec/bouncer` library (just wait a short moment for the first docker image build)
 
 ```bash
-docker-compose run app composer init --no-interaction --require crowdsec/bouncer:^0
+docker-compose run app composer init --no-interaction
+docker-compose run app composer require crowdsec/bouncer
 docker-compose run app composer install
 ```
 
@@ -160,7 +161,7 @@ docker-compose run app php check-ip.php 2.3.4.5
 
 For this IP, the cache system will never ask LAPI anymore for the duration of the decision.
 
-Note: By default, a "bypass" decision is stored in the cache for 10 min. You can change this duration while instantiating the library.
+Note: By default, a "bypass" decision is stored in the cache for 1 min. You can change this duration while instantiating the library.
 
 Don't forget to restart the crowdsec container before continuing :-)
 
@@ -410,7 +411,7 @@ docker-compose run app php check-ip.php 2.3.4.5
 
 > Even if CrowdSec LAPI is down, your bouncer can get the correct information.
 
-To stay protected, you have to call the **refresh-cache.php** script periodically (ie each 15minutes). To do so you can use `crontab` like systems but we will not see that in this guide.
+To stay protected, you have to call the **refresh-cache.php** script periodically (ie each 30seconds). To do so you can use `crontab` like systems but we will not see that in this guide.
 
 ## Enjoy implementing nice PHP bouncers!
 

docs/configuration-reference.md

@@ -20,8 +20,9 @@
        // Optional. Cap the remediation to the selected one. Select from 'bypass' (minimum remediation), 'captcha' or 'ban' (maximum remediation). Defaults to 'ban'.
        'max_remediation_level'=> 'ban',
 
-       // Optional. Set the duration we keep in cache the fact that an IP is clean. In seconds. Defaults to 600 (10 minutes).
-       'cache_expiration_for_clean_ip'=> '600',
+
+       // Optional. Set the duration we keep in cache the fact that an IP is clean. In seconds. Defaults to 60 (1 minute).
+       'cache_expiration_for_clean_ip'=> '60',
    ]
    $cacheAdapter = (...)
    $bouncer = new Bouncer();

src/ApiCache.php

@@ -402,8 +402,9 @@ public function get(string $ip): string
         }
 
         if ($this->adapter->hasItem($ip)) {
-            $this->logger->debug("Cache hit for IP: $ip");
-            return $this->hit($ip);
+            $remediation = $this->hit($ip);
+            $this->logger->debug("Cache hit for IP: $ip: $remediation");
+            return $remediation;
         } else {
             $this->logger->debug("Cache miss for IP: $ip");
             return $this->miss($ip);

src/Bouncer.php

@@ -135,8 +135,6 @@ public function pruneCache(): bool
         return $this->apiCache->clear();
     }
 
-    
-
     /**
      * Browse the remediations cache.
      */

src/Constants.php

@@ -24,8 +24,8 @@ class Constants
     /** @var int The timeout when calling LAPI or CAPI */
     public const API_TIMEOUT = 1;
 
-    /** @var int The duration we keep a clean IP in cache 600s = 10m */
-    public const CACHE_EXPIRATION_FOR_CLEAN_IP = 600; // TODO P3 check the correct bypass duration
+    /** @var int The duration we keep a clean IP in cache 60s = 1m */
+    public const CACHE_EXPIRATION_FOR_CLEAN_IP = 60;
 
     /** @var string The ban remediation */
     public const REMEDIATION_BAN = 'ban';