Check availability of userAgent and acceptHeader before assigning (#682)

ea67f3ca · cyattilakiss · GitHub · 61822472 · ea67f3ca
Commit ea67f3ca authored Apr 01, 2020 by cyattilakiss Committed by GitHub Apr 01, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 5 deletions

Helper/Requests.php Helper/Requests.php +8 -5

No files found.
--- a/Helper/Requests.php
+++ b/Helper/Requests.php
@@ -288,10 +288,13 @@ class Requests extends AbstractHelper
     */
    public function buildBrowserData($request = [])
    {
-        $request['browserInfo'] = [
-            'userAgent' => $_SERVER['HTTP_USER_AGENT'],
-            'acceptHeader' => $_SERVER['HTTP_ACCEPT']
-        ];
+        if (!empty($_SERVER['HTTP_USER_AGENT'])) {
+            $request['browserInfo']['userAgent'] = $_SERVER['HTTP_USER_AGENT'];
+        }
+
+        if (!empty($_SERVER['HTTP_ACCEPT'])) {
+            $request['browserInfo']['acceptHeader'] = $_SERVER['HTTP_ACCEPT'];
+        }

        return $request;
    }
@@ -324,7 +327,7 @@ class Requests extends AbstractHelper
            $request['origin'] = $this->adyenHelper->getOrigin();
            $request['channel'] = 'web';
        }
-        
+
        return $request;
    }