Don't double-escape query params

Author: JordonPhillips

packages/smithy-http/src/smithy_http/serializers.py

@@ -328,7 +328,7 @@ def __init__(
             :py:class:`HTTPHeaderTrait` will be checked instead. Required when
             collecting list entries.
         :param headers: An optional list of header tuples to append to. If not
-            set, one will be created.
+            set, one will be created. Values appended will not be escaped.
         """
         self.headers: list[tuple[str, str]] = headers if headers is not None else []
         self._key = key
@@ -486,7 +486,7 @@ def write_big_decimal(self, schema: Schema, value: Decimal) -> None:
 
     def write_string(self, schema: Schema, value: str) -> None:
         key = self._key or schema.expect_trait(HTTPQueryTrait).key
-        self.query_params.append((key, urlquote(value, safe="")))
+        self.query_params.append((key, value))
 
     def write_timestamp(self, schema: Schema, value: datetime) -> None:
         key = self._key or schema.expect_trait(HTTPQueryTrait).key
@@ -576,7 +576,7 @@ def entry(self, key: str, value_writer: Callable[[ShapeSerializer], None]):
 
 
 class HTTPQueryMapValueSerializer(SpecificShapeSerializer):
-    def __init__(self, key: str,  query_params: list[tuple[str, str]]) -> None:
+    def __init__(self, key: str, query_params: list[tuple[str, str]]) -> None:
         """Initialize an HTTPQueryMapValueSerializer.
 
         :param key: The key of the query parameter.
@@ -586,7 +586,8 @@ def __init__(self, key: str,  query_params: list[tuple[str, str]]) -> None:
         self._query_params = query_params
 
     def write_string(self, schema: Schema, value: str) -> None:
-        self._query_params.append((self._key, urlquote(value, safe="")))
+        # Note: values are escaped when query params are joined
+        self._query_params.append((self._key, value))
 
     @contextmanager
     def begin_list(self, schema: Schema, size: int) -> Iterator[ShapeSerializer]: