SECURITY-480: build fix

Co-authored-by: Laszlo Losonczy <laszlo.losonczy@emarsys.com>
emartech · Jun 14, 2024 · ed14aff · ed14aff
1 parent 08c101f
commit ed14aff
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/Makefile b/Makefile
@@ -5,6 +5,6 @@ build: ; docker compose build
 install: ; docker compose run --rm web composer install
 update: ; docker compose run --rm web composer update
 
-test: ; docker compose run --rm web composer test
+test: ; docker compose run --rm web php -d error_reporting=E_ALL ./vendor/bin/phpunit --do-not-cache-result -c phpunit.xml
 test-only: ; docker compose run --rm web ./vendor/bin/phpunit --do-not-cache-result --group only -c phpunit.xml
 
diff --git a/src/Escher/RequestHelper.php b/src/Escher/RequestHelper.php
@@ -51,16 +51,15 @@ public function getHeaderList()
         $headerList = $this->process($this->serverVars);
         $headerList['content-type'] = $this->getContentType();
 
-        if (strpos($headerList['host'], ':') === false)
-        {
-            $host = $headerList['host'];
-            $port = null;
-        }
-        else
-        {
-            list($host, $port) = explode(':', $headerList['host'], 2);
+        if (isset($headerList['host'])) {
+            if (strpos($headerList['host'], ':') === false) {
+                $host = $headerList['host'];
+                $port = null;
+            } else {
+                list($host, $port) = explode(':', $headerList['host'], 2);
+            }
+            $headerList['host'] = $this->normalizeHost($host, $port);
         }
-        $headerList['host'] = $this->normalizeHost($host, $port);
 
         return $headerList;
     }