Merge pull request #173 from utopia-php/fix-null-path-1.x

Fix: issue with matching null path and relevant test

Author: lohanidamodar

src/Http/Http.php

@@ -383,6 +383,11 @@ public function start()
     public function match(Request $request): ?Route
     {
         $url = \parse_url($request->getURI(), PHP_URL_PATH);
+
+        if ($url === null || $url === false) {
+            $url = '/'; // Default to root path for malformed URLs
+        }
+
         $method = $request->getMethod();
         $method = (self::REQUEST_METHOD_HEAD == $method) ? self::REQUEST_METHOD_GET : $method;
 

tests/HttpTest.php

@@ -608,6 +608,58 @@ public function testCanMatchRoute(string $method, string $path, ?string $url = n
         $this->assertEquals($expected, $route);
     }
 
+    public function testMatchWithNullPath(): void
+    {
+        // Create a route for root path
+        $expected = Http::get('/');
+
+        // Test case where parse_url returns null (malformed URL)
+        $_SERVER['REQUEST_METHOD'] = 'GET';
+        $_SERVER['REQUEST_URI'] = '?param=1'; // This will cause parse_url to return null for PATH component
+
+        $matched = $this->http->match(new Request());
+        $this->assertEquals($expected, $matched);
+    }
+
+    public function testMatchWithEmptyPath(): void
+    {
+        // Create a route for root path
+        $expected = Http::get('/');
+
+        // Test case where URI has no path component
+        $_SERVER['REQUEST_METHOD'] = 'GET';
+        $_SERVER['REQUEST_URI'] = 'https://example.com'; // No path component
+
+        $matched = $this->http->match(new Request());
+        $this->assertEquals($expected, $matched);
+    }
+
+    public function testMatchWithMalformedURL(): void
+    {
+        // Create a route for root path
+        $expected = Http::get('/');
+
+        // Test case where parse_url returns false (severely malformed URL)
+        $_SERVER['REQUEST_METHOD'] = 'GET';
+        $_SERVER['REQUEST_URI'] = '#fragment'; // Malformed scheme
+
+        $matched = $this->http->match(new Request());
+        $this->assertEquals($expected, $matched);
+    }
+
+    public function testMatchWithOnlyQueryString(): void
+    {
+        // Create a route for root path
+        $expected = Http::get('/');
+
+        // Test case where URI has only query string (no path)
+        $_SERVER['REQUEST_METHOD'] = 'GET';
+        $_SERVER['REQUEST_URI'] = '?param=value'; // Only query string, no path
+
+        $matched = $this->http->match(new Request());
+        $this->assertEquals($expected, $matched);
+    }
+
     public function testNoMismatchRoute(): void
     {
         $requests = [