terrylinooo / shieldon

src/Firewall/Firewall/SetupTrait.php

Spacing +6 added lines, -6 removed lines

@@ -314,7 +314,7 @@ 
 
         foreach ($ipList as $ip) {
 
-            if (0 === strpos($this->kernel->getCurrentUrl(), $ip['url']) ) {
+            if (0 === strpos($this->kernel->getCurrentUrl(), $ip['url'])) {
 
                 if ('allow' === $ip['rule']) {
                     $allowedList[] = $ip['ip'];
@@ -376,7 +376,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   
             ]
         );
@@ -384,7 +384,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
             ]
         );
@@ -429,8 +429,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);
@@ -453,7 +453,7 @@ 
 
             $lastResetTime = $cronjobSetting['config']['last_update'];
 
-            if (!empty($lastResetTime) ) {
+            if (!empty($lastResetTime)) {
                 $lastResetTime = strtotime($lastResetTime);
             } else {
                 // @codeCoverageIgnoreStart

src/Firewall/Log/ActionLogParser.php

Spacing +26 added lines, -26 removed lines

@@ -207,7 +207,7 @@ 
         ];
 
         if (empty($dataRange[$this->type])) {
-            if (preg_match('/past_([0-9]+)_days/', $this->type, $matches) ) {
+            if (preg_match('/past_([0-9]+)_days/', $this->type, $matches)) {
                 $dayCount = $matches[1];
                 $startDate = date('Ymd', strtotime('-' . $dayCount . ' days'));
                 $endDate = date('Ymd');
@@ -350,18 +350,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();
 
@@ -406,18 +406,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)) {
 
@@ -477,7 +477,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']++;