Improve URL handling with support for trailing slashes and special characters

  This PR provides a more consistent approach to URL handling in Inertia responses by:

  - Adding support for preserving trailing slashes in URLs
  - Improving readability by decoding special characters in query parameters
  - Maintaining compatibility with proxy prefixes
  - Ensuring URL consistency for SEO and debugging

  Previously, characters like slashes (%2F) and ampersands (%26) were encoded in the URL,
  making debugging more difficult and causing inconsistencies between browser URLs and
  Inertia history state URLs.

  Comprehensive tests have been added for both trailing and non-trailing slash scenarios,
  as well as specific tests for slash and ampersand handling in query parameters.

  Resolves issues discussed in inertiajs#663

Author: ljbw

src/Response.php

@@ -365,15 +365,24 @@ public function isPartial(Request $request): bool
      */
     protected function getRequestUrl(Request $request): string
     {
-        $uri = $request->getRequestUri();
+        // Get the original request URI which should already contain any base path
+        $originalUri = $request->getRequestUri();
 
         // Handle X-Forwarded-Prefix header for proxy setups
         if ($prefix = $request->header('X_FORWARDED_PREFIX')) {
-            if (!str_starts_with($uri, $prefix)) {
-                $uri = $prefix . $uri;
+            // Only add the prefix if it's not already at the start of the URI
+            if (!str_starts_with($originalUri, $prefix)) {
+                $originalUri = $prefix . $originalUri;
             }
         }
 
-        return $uri;
+        // Decode the query string portion for better readability
+        if (str_contains($originalUri, '?')) {
+            list($path, $query) = explode('?', $originalUri, 2);
+            $query = urldecode($query);
+            return $path . '?' . $query;
+        }
+        
+        return $originalUri;
     }
 }

tests/ResponseTest.php

@@ -32,6 +32,18 @@ public function test_can_macro(): void
         $this->assertEquals('bar', $response->foo());
     }
 
+public function test_the_page_url_is_not_encoded(): void
+    {
+        $request = Request::create('/product/123', 'GET', ['value' => 'te/st']);
+        $request->headers->add(['X-Inertia' => 'true']);
+
+        $response = new Response('Product/Show', []);
+        $response = $response->toResponse($request);
+        $page = $response->getData();
+
+        $this->assertSame('/product/123?value=te/st', $page->url);
+    }
+
     public function test_server_response(): void
     {
         $request = Request::create('/user/123', 'GET');
@@ -895,4 +907,44 @@ public function test_non_trailing_slashes_in_url_work_correctly(): void
 
         $this->assertSame('/admin/categories', $page->url);
     }
+    
+    public function test_url_with_slashes_in_query_params(): void
+    {
+        // Test with forward slashes in query parameters
+        $request = Request::create('/product/123?value=te/st', 'GET');
+        $request->headers->add(['X-Inertia' => 'true']);
+        
+        $response = new Response('Product/Show', []);
+        $response = $response->toResponse($request);
+        $page = $response->getData();
+        
+        // Forward slashes should be preserved, not encoded as %2F
+        $this->assertSame('/product/123?value=te/st', $page->url);
+    }
+    
+    public function test_url_with_ampersands_in_query_params(): void
+    {
+        // Test normal query parameters with & as separator
+        $request = Request::create('/families?search=jonathan&amy=test', 'GET');
+        $request->headers->add(['X-Inertia' => 'true']);
+        
+        $response = new Response('Family/Index', []);
+        $response = $response->toResponse($request);
+        $page = $response->getData();
+        
+        // Check that the URL contains both parameters properly separated by &
+        $this->assertStringContainsString('search=jonathan', $page->url);
+        $this->assertStringContainsString('amy=test', $page->url);
+        
+        // Test encoded ampersand within a parameter value
+        $request = Request::create('/families?search=jonathan%26amy', 'GET');
+        $request->headers->add(['X-Inertia' => 'true']);
+        
+        $response = new Response('Family/Index', []);
+        $response = $response->toResponse($request);
+        $page = $response->getData();
+        
+        // The & should be decoded in the parameter value
+        $this->assertSame('/families?search=jonathan&amy', $page->url);
+    }
 }