Merge pull request #1 from Skywire/arrayFix

valguss · web-flow · commit f4a5d35493a9 · 2021-07-22T09:56:00.000+01:00
DD#0000: fix: Added check around array values
diff --git a/src/Processor/UrlProcessor.php b/src/Processor/UrlProcessor.php
@@ -17,7 +17,9 @@ class UrlProcessor implements ProcessorInterface
 {
     public function __invoke(array $records)
     {
-        $records['extra']['url'] = $_SERVER['REQUEST_METHOD'] . ' ' . $_SERVER['REQUEST_URI'];
+        if (isset($_SERVER['REQUEST_METHOD']) && isset($_SERVER['REQUEST_URI'])) {
+            $records['extra']['url'] = $_SERVER['REQUEST_METHOD'] . ' ' . $_SERVER['REQUEST_URI'];
+        }
 
         return $records;
     }

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,9 @@ class UrlProcessor implements ProcessorInterface`
`17`	`17`	`{`
`18`	`18`	`public function __invoke(array $records)`
`19`	`19`	`{`
`20`		`- $records['extra']['url'] = $_SERVER['REQUEST_METHOD'] . ' ' . $_SERVER['REQUEST_URI'];`
	`20`	`+ if (isset($_SERVER['REQUEST_METHOD']) && isset($_SERVER['REQUEST_URI'])) {`
	`21`	`+ $records['extra']['url'] = $_SERVER['REQUEST_METHOD'] . ' ' . $_SERVER['REQUEST_URI'];`
	`22`	`+ }`
`21`	`23`
`22`	`24`	`return $records;`
`23`	`25`	`}`