smithy-http 0.4.3__tar.gz → 0.4.4__tar.gz

smithy_http-0.4.4/.changes/0.4.4.json

@@ -0,0 +1,16 @@
+{
+  "changes": [
+    {
+      "type": "enhancement",
+      "description": "Added a protocol hook for extracting server-provided retry delays from error responses."
+    },
+    {
+      "type": "enhancement",
+      "description": "Declared `AWSCRTHTTPClient` as duplex (bidirectional) streaming capable via `SUPPORTS_DUPLEX_STREAMING`. `AIOHTTPClient` explicitly does not support duplex streaming."
+    },
+    {
+      "type": "enhancement",
+      "description": "Disabled automatic redirects in `AIOHTTPClient` so that redirect responses are returned to the caller instead of being followed transparently."
+    }
+  ]
+}

{smithy_http-0.4.3 → smithy_http-0.4.4}/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## v0.4.4
+
+### Enhancements
+* Added a protocol hook for extracting server-provided retry delays from error responses.
+* Declared `AWSCRTHTTPClient` as duplex (bidirectional) streaming capable via `SUPPORTS_DUPLEX_STREAMING`. `AIOHTTPClient` explicitly does not support duplex streaming.
+* Disabled automatic redirects in `AIOHTTPClient` so that redirect responses are returned to the caller instead of being followed transparently.
+
 ## v0.4.3
 
 ### Enhancements

{smithy_http-0.4.3 → smithy_http-0.4.4}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.5
 Name: smithy-http
-Version: 0.4.3
+Version: 0.4.4
 Summary: HTTP components for Smithy tooling.
 Project-URL: Changelog, https://github.com/smithy-lang/smithy-python/blob/develop/packages/smithy-http/CHANGELOG.md
 Project-URL: Code, https://github.com/smithy-lang/smithy-python/blob/develop/packages/smithy-http/

{smithy_http-0.4.3 → smithy_http-0.4.4}/src/smithy_http/__init__.py

@@ -7,7 +7,7 @@ from typing import get_args
 from . import interfaces
 from .interfaces import FieldPosition
 
-__version__ = "0.4.3"
+__version__ = "0.4.4"
 
 _VALID_FIELD_POSITIONS = frozenset(get_args(FieldPosition))
 

{smithy_http-0.4.3 → smithy_http-0.4.4}/src/smithy_http/aio/aiohttp.py

@@ -53,6 +53,10 @@ class AIOHTTPClient(HTTPClient):
 
     TIMEOUT_EXCEPTIONS = (TimeoutError,)
 
+    # aiohttp has no HTTP/2 support and this client fully buffers the response
+    # before returning, so it can never interleave request and response data.
+    SUPPORTS_DUPLEX_STREAMING = False
+
     def __init__(
         self,
         *,
@@ -108,6 +112,7 @@ class AIOHTTPClient(HTTPClient):
             params=parse_qs(request.destination.query),  # type: ignore
             headers=headers_list,
             data=body,
+            allow_redirects=False,
         ) as resp:
             return await self._marshal_response(resp)
 

{smithy_http-0.4.3 → smithy_http-0.4.4}/src/smithy_http/aio/crt.py

@@ -142,6 +142,10 @@ class AWSCRTHTTPClient(http_aio_interfaces.HTTPClient):
 
     TIMEOUT_EXCEPTIONS = (_CRTTimeoutError,)
 
+    # CRT can stream request and response data concurrently when the connection
+    # negotiates HTTP/2.
+    SUPPORTS_DUPLEX_STREAMING = True
+
     def __init__(
         self,
         eventloop: _AWSCRTEventLoop | None = None,

{smithy_http-0.4.3 → smithy_http-0.4.4}/src/smithy_http/aio/protocols.py

@@ -192,6 +192,8 @@ class HttpBindingClientProtocol(HttpClientProtocol):
                 error_id=error_id,
             )
 
+        retry_after = self._retry_after(response)
+
         if error_id is None and self._matches_content_type(response):
             if isinstance(response_body, bytearray):
                 response_body = bytes(response_body)
@@ -225,7 +227,10 @@ class HttpBindingClientProtocol(HttpClientProtocol):
                 response=response,
                 body=response_body,
             )
-            return error_shape.deserialize(deserializer)
+            modeled_error = error_shape.deserialize(deserializer)
+            if retry_after is not None:
+                modeled_error.retry_after = retry_after
+            return modeled_error
 
         message = (
             f"Unknown error for operation {operation.schema.id} "
@@ -246,8 +251,13 @@ class HttpBindingClientProtocol(HttpClientProtocol):
             is_throttling_error=is_throttle,
             is_timeout_error=is_timeout,
             is_retry_safe=is_throttle or is_timeout or None,
+            retry_after=retry_after,
         )
 
+    def _retry_after(self, response: HTTPResponse) -> float | None:
+        """The retry delay in seconds requested by the server, if the response carries one."""
+        return None
+
     def _resolve_error_id(
         self,
         *,

{smithy_http-0.4.3 → smithy_http-0.4.4}/src/smithy_http/testing/mockhttp.py

@@ -22,6 +22,11 @@ class MockHTTPClient(HTTPClient):
 
     TIMEOUT_EXCEPTIONS = (TimeoutError,)
 
+    # Declared so duplex (bidirectional) stream operations can be unit tested
+    # against this client. Queued responses are returned without consuming the
+    # request body, which is compatible with duplex streaming.
+    SUPPORTS_DUPLEX_STREAMING = True
+
     def __init__(
         self,
         *,