terrylinooo / shieldon

src/Firewall/Component/Rdns.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     public function isDenied(): bool
     {
         if (!empty($this->deniedList)) {
-            if (preg_match('/(' . implode('|', $this->deniedList). ')/i', $this->rdns)) {
+            if (preg_match('/(' . implode('|', $this->deniedList) . ')/i', $this->rdns)) {
                 return true;
             }
         }

src/Firewall/Component/UserAgent.php

Spacing +10 added lines, -10 removed lines

@@ -49,19 +49,19 @@ 
         $this->deniedList = [
 
             // Backlink crawlers
-            'Ahrefs',     // http://ahrefs.com/robot/
-            'roger',      // rogerbot (SEOMOZ)
-            'moz.com',    // SEOMOZ crawlers
-            'MJ12bot',    // Majestic crawlers
-            'findlinks',  // http://wortschatz.uni-leipzig.de/findlinks
-            'Semrush',    // http://www.semrush.com/bot.html
+            'Ahrefs', // http://ahrefs.com/robot/
+            'roger', // rogerbot (SEOMOZ)
+            'moz.com', // SEOMOZ crawlers
+            'MJ12bot', // Majestic crawlers
+            'findlinks', // http://wortschatz.uni-leipzig.de/findlinks
+            'Semrush', // http://www.semrush.com/bot.html
     
             // Web information crawlers
-            'domain',     // Domain name information crawlers.
-            'copyright',  // Copyright information crawlers.
+            'domain', // Domain name information crawlers.
+            'copyright', // Copyright information crawlers.
     
             // Others
-            'archive',    // Wayback machine
+            'archive', // Wayback machine
         ];
     }
 
@@ -71,7 +71,7 @@ 
     public function isDenied(): bool
     {
         if (!empty($this->deniedList)) {
-            if (preg_match('/(' . implode('|', $this->deniedList). ')/i', $this->userAgent)) {
+            if (preg_match('/(' . implode('|', $this->deniedList) . ')/i', $this->userAgent)) {
                 return true;
             }
         }

src/Firewall/Middleware/UserAgent.php

Spacing +10 added lines, -10 removed lines

@@ -50,19 +50,19 @@ 
     protected $deniedList = [
 
         // Backlink crawlers
-        'Ahrefs',     // http://ahrefs.com/robot/
-        'roger',      // rogerbot (SEOMOZ)
-        'moz.com',    // SEOMOZ crawlers
-        'MJ12bot',    // Majestic crawlers
-        'findlinks',  // http://wortschatz.uni-leipzig.de/findlinks
-        'Semrush',    // http://www.semrush.com/bot.html
+        'Ahrefs', // http://ahrefs.com/robot/
+        'roger', // rogerbot (SEOMOZ)
+        'moz.com', // SEOMOZ crawlers
+        'MJ12bot', // Majestic crawlers
+        'findlinks', // http://wortschatz.uni-leipzig.de/findlinks
+        'Semrush', // http://www.semrush.com/bot.html
 
         // Web information crawlers
-        'domain',     // Domain name information crawlers.
-        'copyright',  // Copyright information crawlers.
+        'domain', // Domain name information crawlers.
+        'copyright', // Copyright information crawlers.
 
         // Others
-        'archive',    // Wayback machine
+        'archive', // Wayback machine
     ];
 
     /**
@@ -96,7 +96,7 @@ 
         }
 
         if (!empty($this->deniedList)) {
-            if (preg_match('/(' . implode('|', $this->deniedList). ')/i', $userAgent)) {
+            if (preg_match('/(' . implode('|', $this->deniedList) . ')/i', $userAgent)) {
                 return (new Response)->withStatus(self::HTTP_STATUS_CODE);
             }
         }

src/Firewall/Driver/RedisDriver.php

Spacing +2 added lines, -2 removed lines

@@ -274,8 +274,8 @@
     {
         $table = [
             'filter'  => $this->tableFilterLogs . ':' . $ip,
-            'session' => $this->tableSessions   . ':' . $ip,
-            'rule'    => $this->tableRuleList   . ':' . $ip,
+            'session' => $this->tableSessions . ':' . $ip,
+            'rule'    => $this->tableRuleList . ':' . $ip,
         ];
         
         return $table[$type] ?? '';

src/Firewall/Driver/FileDriverTrait.php

Spacing +2 added lines, -2 removed lines

@@ -100,8 +100,8 @@
         $path = [];
 
         $path['filter']  = $this->directory . '/' . $this->tableFilterLogs . '/' . $ip . '.' . $this->extension;
-        $path['session'] = $this->directory . '/' . $this->tableSessions   . '/' . $ip . '.' . $this->extension;
-        $path['rule']    = $this->directory . '/' . $this->tableRuleList   . '/' . $ip . '.' . $this->extension;
+        $path['session'] = $this->directory . '/' . $this->tableSessions . '/' . $ip . '.' . $this->extension;
+        $path['rule']    = $this->directory . '/' . $this->tableRuleList . '/' . $ip . '.' . $this->extension;
 
         return $path[$type] ?? '';
     }

src/Firewall/FirewallTrait.php

Spacing +1 added lines, -1 removed lines

@@ -272,7 +272,7 @@
             $channel = '_' . $this->channel;
         }
 
-        return $this->directory . '/' . $this->version . $channel .  '_' . $driverType . '_' . $this->checkpoint;
+        return $this->directory . '/' . $this->version . $channel . '_' . $driverType . '_' . $this->checkpoint;
     }
 
     /**

src/Firewall/Firewall/SetupTrait.php

Spacing +8 added lines, -8 removed lines

@@ -316,7 +316,7 @@ 
             if (empty($ip))
                 continue;
 
-            if (0 === strpos($this->kernel->getCurrentUrl(), empty($ip['url']) ? '/' : $ip['url']) ) {
+            if (0 === strpos($this->kernel->getCurrentUrl(), empty($ip['url']) ? '/' : $ip['url'])) {
 
                 if ('allow' === $ip['rule']) {
                     $allowedList[] = $ip['ip'];
@@ -378,7 +378,7 @@ 
         $this->kernel->setProperty(
             'deny_attempt_enable',
             [
-                'data_circle'     => $setting['data_circle']['enable'],     // false   
+                'data_circle'     => $setting['data_circle']['enable'], // false   
                 'system_firewall' => $setting['system_firewall']['enable'], // false   
             ]
         );
@@ -386,7 +386,7 @@ 
         $this->kernel->setProperty(
             'deny_attempt_buffer',
             [
-                'data_circle'     => $setting['data_circle']['buffer'],     // 10
+                'data_circle'     => $setting['data_circle']['buffer'], // 10
                 'system_firewall' => $setting['system_firewall']['buffer'], // 10
             ]
         );
@@ -431,8 +431,8 @@ 
 
         if ($sessionLimitSetting['enable']) {
 
-            $onlineUsers = $sessionLimitSetting['config']['count'];       // default: 100
-            $alivePeriod = $sessionLimitSetting['config']['period'];      // default: 300
+            $onlineUsers = $sessionLimitSetting['config']['count']; // default: 100
+            $alivePeriod = $sessionLimitSetting['config']['period']; // default: 300
             $isUniqueIp  = $sessionLimitSetting['config']['unique_only']; // false
 
             $this->kernel->limitSession($onlineUsers, $alivePeriod, $isUniqueIp);
@@ -455,7 +455,7 @@ 
 
             $lastResetTime = $cronjobSetting['config']['last_update'];
 
-            if (!empty($lastResetTime) ) {
+            if (!empty($lastResetTime)) {
                 $lastResetTime = strtotime($lastResetTime);
             } else {
                 // @codeCoverageIgnoreStart
@@ -535,8 +535,8 @@ 
         $dialogInfo = $this->getOption('dialog_info_disclosure');
 
         $this->kernel->setProperty('display_online_info', $dialogInfo['online_user_amount']);
-        $this->kernel->setProperty('display_user_info',   $dialogInfo['user_inforamtion']);
-        $this->kernel->setProperty('display_http_code',   $dialogInfo['http_status_code']);
+        $this->kernel->setProperty('display_user_info', $dialogInfo['user_inforamtion']);
+        $this->kernel->setProperty('display_http_code', $dialogInfo['http_status_code']);
         $this->kernel->setProperty('display_reason_code', $dialogInfo['reason_code']);
         $this->kernel->setProperty('display_reason_text', $dialogInfo['reason_text']);
     }

Braces +3 added lines, -2 removed lines

@@ -313,8 +313,9 @@
         $deniedList = [];
 
         foreach ($ipList as $ip) {
-            if (empty($ip))
-                continue;
+            if (empty($ip)) {
+                            continue;
+            }
 
             if (0 === strpos($this->kernel->getCurrentUrl(), empty($ip['url']) ? '/' : $ip['url']) ) {
 

src/Firewall/Log/ActionLogParser.php

Spacing +25 added lines, -25 removed lines

@@ -347,18 +347,18 @@ 
 
         $results['captcha_chart_string']       = ''; // string
         $results['pageview_chart_string']      = ''; // string
-        $results['captcha_success_count']      = 0;  // integer
-        $results['captcha_failure_count']      = 0;  // integer
-        $results['captcha_count']              = 0;  // integer
-        $results['pageview_count']             = 0;  // integer
-        $results['captcha_percentageage']      = 0;  // integer
-        $results['captcha_failure_percentage'] = 0;  // integer
-        $results['captcha_success_percentage'] = 0;  // integer
-        $results['action_ban_count']           = 0;  // integer
-        $results['action_temp_ban_count']      = 0;  // integer
-        $results['action_unban_count']         = 0;  // integer
-        $results['blacklist_count']            = 0;  // integer
-        $results['session_limit_count']        = 0;  // integer
+        $results['captcha_success_count']      = 0; // integer
+        $results['captcha_failure_count']      = 0; // integer
+        $results['captcha_count']              = 0; // integer
+        $results['pageview_count']             = 0; // integer
+        $results['captcha_percentageage']      = 0; // integer
+        $results['captcha_failure_percentage'] = 0; // integer
+        $results['captcha_success_percentage'] = 0; // integer
+        $results['action_ban_count']           = 0; // integer
+        $results['action_temp_ban_count']      = 0; // integer
+        $results['action_unban_count']         = 0; // integer
+        $results['blacklist_count']            = 0; // integer
+        $results['session_limit_count']        = 0; // integer
 
         $ipdData = $this->getIpData();
 
@@ -407,18 +407,18 @@ 
         $results['label_chart_string']         = ''; // string
         $results['captcha_chart_string']       = ''; // string
         $results['pageview_chart_string']      = ''; // string
-        $results['captcha_success_count']      = 0;  // integer
-        $results['captcha_failure_count']      = 0;  // integer
-        $results['captcha_count']              = 0;  // integer
-        $results['pageview_count']             = 0;  // integer
-        $results['captcha_percentageage']      = 0;  // integer
-        $results['captcha_failure_percentage'] = 0;  // integer
-        $results['captcha_success_percentage'] = 0;  // integer
-        $results['action_ban_count']           = 0;  // integer
-        $results['action_temp_ban_count']      = 0;  // integer
-        $results['action_unban_count']         = 0;  // integer
-        $results['blacklist_count']            = 0;  // integer
-        $results['session_limit_count']        = 0;  // integer
+        $results['captcha_success_count']      = 0; // integer
+        $results['captcha_failure_count']      = 0; // integer
+        $results['captcha_count']              = 0; // integer
+        $results['pageview_count']             = 0; // integer
+        $results['captcha_percentageage']      = 0; // integer
+        $results['captcha_failure_percentage'] = 0; // integer
+        $results['captcha_success_percentage'] = 0; // integer
+        $results['action_ban_count']           = 0; // integer
+        $results['action_temp_ban_count']      = 0; // integer
+        $results['action_unban_count']         = 0; // integer
+        $results['blacklist_count']            = 0; // integer
+        $results['session_limit_count']        = 0; // integer
 
         if (!empty($periodData)) {
             $chartCaptcha = [];
@@ -483,7 +483,7 @@ 
         $ip = $log['ip'];
         $sessionId = $log['session_id'];
 
-        $this->ipDetail[$t][$ip]['session_id'][$sessionId ] = 1;
+        $this->ipDetail[$t][$ip]['session_id'][$sessionId] = 1;
 
         if ($logActionCode === self::LOG_TEMPORARILY_BAN) {
             $this->periodDetail[$t][$k]['action_temp_ban_count']++;

src/Firewall/Firewall/XssProtectionTrait.php

Spacing +4 added lines, -4 removed lines

@@ -87,7 +87,7 @@ 
         if ($enable['post']) {
             $this->kernel->setClosure(
                 'xss_post',
-                function () use ($xss) {
+                function() use ($xss) {
                     if (!empty($_POST)) {
                         foreach (array_keys($_POST) as $k) {
                             $_POST[$k] = $xss->clean($_POST[$k]);
@@ -111,7 +111,7 @@ 
         if ($enable['get']) {
             $this->kernel->setClosure(
                 'xss_get',
-                function () use ($xss) {
+                function() use ($xss) {
                     if (!empty($_GET)) {
                         foreach (array_keys($_GET) as $k) {
                             $_GET[$k] = $xss->clean($_GET[$k]);
@@ -135,7 +135,7 @@ 
         if ($enable['cookie']) {
             $this->kernel->setClosure(
                 'xss_cookie',
-                function () use ($xss) {
+                function() use ($xss) {
                     if (!empty($_COOKIE)) {
                         foreach (array_keys($_COOKIE) as $k) {
                             $_COOKIE[$k] = $xss->clean($_COOKIE[$k]);
@@ -159,7 +159,7 @@ 
         if (!empty($protectedList)) {
             $this->kernel->setClosure(
                 'xss_protection',
-                function () use ($xss, $protectedList) {
+                function() use ($xss, $protectedList) {
                     foreach ($protectedList as $v) {
                         $k = $v['variable'] ?? 'undefined';