Configurable log stack

Author: Bruno Cabral

src/Jobs/CheckIfBotIsReal.php

@@ -71,7 +71,7 @@ public function handle()
             Redis::sadd($key_whitelist, $ip);
 
             if ($log_blocked_requests){
-                Log::info("[Block-Bots] IP: {$ip} with User agent: {$user_agent} was detected as a GOOD crawler");
+                Log::stack($this->config['channels_info'])->info("[Block-Bots] IP: {$ip} with User agent: {$user_agent} was detected as a GOOD crawler");
             }
 
         }
@@ -80,7 +80,7 @@ public function handle()
             Redis::sadd($key_fake_bot, $ip);
 
             if ($log_blocked_requests){
-                Log::info("[Block-Bots] IP: {$ip} with User agent: {$user_agent} was detected as a BAD crawler");
+                Log::stack($this->config['channels_info'])->info("[Block-Bots] IP: {$ip} with User agent: {$user_agent} was detected as a BAD crawler");
             }
         }
 

src/Middleware/BlockBots.php

@@ -31,7 +31,7 @@ public function __construct()
      */
     public function handle($request, Closure $next, $dailyLimit)
     {
-        $enabled = $this->config['fake_mode'];
+        $enabled = $this->config['enabled'];
         if (!$enabled){
             return $next($request);
         }
@@ -40,7 +40,7 @@ public function handle($request, Closure $next, $dailyLimit)
             $blocked = $this->blocked($request, $dailyLimit);
 
         } catch (Exception $e) {
-            Log::error("[Block-Bots] Error at handling request: {$e->getMessage()}");
+            Log::stack($this->config['channels_info'])->error("[Block-Bots] Error at handling request: {$e->getMessage()}");
             $blocked = false;
         }
 
@@ -62,6 +62,7 @@ public function handle($request, Closure $next, $dailyLimit)
     public function blocked($request, $dailyLimit)
     {
 
+        $dailyLimit = (int) $dailyLimit;
         $ip = $request->getClientIp();
         $user_agent = $request->header('User-Agent');
         $fake_mode = $this->config['fake_mode'];
@@ -94,12 +95,19 @@ public function blocked($request, $dailyLimit)
             return false;
         }
         else{
-            if($fake_mode)
-            {
                 if ($log_blocked_requests){
-                    Log::error("[Block-Bots] IP: {$ip} with User agent: {$user_agent} would be blocked after {$number_of_hits} hits while accessing URL {$full_url}");
+                $key_notified = "block_bot:notified:{$ip}";
+                if(!Redis::exists($key_notified))
+                {
+                    $end_of_day_unix_timestamp = Carbon::tomorrow()->startOfDay()->timestamp;
+                    Redis::set($key_notified, 1);
+                    Redis::expireat($key_notified, $end_of_day_unix_timestamp);
+                    Log::stack($this->config['channels_blocks'])->error("[Block-Bots] IP: {$ip} with User agent: {$user_agent} would be blocked after {$number_of_hits} hits while accessing URL {$full_url}");
+                }
                 }
 
+            if($fake_mode)
+            {
                 return false;
             }
             else{
@@ -142,7 +150,7 @@ public function isWhitelisted($ip, $user_agent)
             //Add this to the redis list as it is faster
             Redis::sadd($key_whitelist, $ip);
             if ($this->config['log_blocked_requests']){
-                Log::info("[Block-Bots] WHITELISTED IP: {$ip} with User agent: {$user_agent} because was on our config");
+                Log::stack($this->config['channels_info'])->info("[Block-Bots] WHITELISTED IP: {$ip} with User agent: {$user_agent} because was on our config");
             }
         }
 

src/config/block-bots.php

@@ -22,6 +22,20 @@
     'allow_logged_user' => env('BLOCK_BOTS_ALLOW_LOGGED_USER', true),
     'fake_mode' => env('BLOCK_BOTS_FAKE_MODE', true), // minutes - disabled by default
 
+    /*
+     * Log channel
+     */
+    'channels_info' => [
+        'single'
+    ],
+
+    /*
+     * This Log channel will receive when a bad crawler is detected or someone is banned
+     */
+    'channels_blocks' => [
+        'single'
+    ],
+
     /*
      * Send suspicious events to log?
      *