fix: preserving dots in query string (#132)

Author: norkunas

src/Browser/HttpOptions.php

@@ -11,6 +11,8 @@
 
 namespace Zenstruck\Browser;
 
+use Symfony\Component\HttpFoundation\HeaderUtils;
+
 /**
  * @author Kevin Bond <[email protected]>
  *
@@ -243,7 +245,7 @@ final public function addQueryToUrl(string $url): string
         }
 
         if (isset($parts['query'])) {
-            \parse_str($parts['query'], $query);
+            $query = HeaderUtils::parseQuery($parts['query']);
         } else {
             $query = [];
         }

tests/HttpOptionsTest.php

@@ -287,4 +287,12 @@ public function can_override_json_and_ajax_headers(): void
             $options->server()
         );
     }
+
+    /**
+     * @test
+     */
+    public function dots_in_query_string_are_preserved(): void
+    {
+        $this->assertSame('/?nested.param=value&deeper.nested.param=y', (new HttpOptions())->addQueryToUrl('/?nested.param=value&deeper.nested.param=y'));
+    }
 }