Merge pull request #575 from favicode/fix/rate-limit-paths

Fix for rate limit paths

Author: vvuksan

Diff for: Controller/Adminhtml/FastlyCdn/RateLimiting/ToggleRateLimiting.php

@@ -278,7 +278,9 @@ private function processPaths()
             $validPaths[] = 'req.url.path ~ "' . $value->path . '"';
         }
 
-        return implode(' || ', $validPaths);
+        return !empty($validPaths)
+            ? implode(' || ', $validPaths)
+            : 'req.url.path ~ "^/fastly-io-tester$"';
     }
 
     /**

Diff for: Controller/Adminhtml/FastlyCdn/RateLimiting/UpdatePaths.php

@@ -114,7 +114,7 @@ public function execute()
             if (!$paths) {
                 $paths = [];
             }
-            $validPaths = '';
+            $validPaths = [];
 
             $snippet = $this->config->getVclSnippets(
                 Config::VCL_RATE_LIMITING_PATH,
@@ -142,10 +142,12 @@ public function execute()
                         'msg'       => $value['path'] . ' is not a valid regular expression'
                     ]);
                 }
-                $validPaths .= 'req.url.path ~ "' . $value['path'] . '" || ';
+                $validPaths[] = 'req.url.path ~ "' . $value['path'] . '"';
             }
 
-            $strippedValidPaths = substr($validPaths, 0, strrpos($validPaths, '||', -1));
+            $strippedValidPaths = !empty($validPaths)
+                ? implode(' || ', $validPaths)
+                : 'req.url.path ~ "^/fastly-io-tester$"';
 
             foreach ($snippet as $key => $value) {
                 if ($validPaths == '') {