strawberry-graphql 0.247.0__tar.gz → 0.247.1__tar.gz

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: strawberry-graphql
-Version: 0.247.0
+Version: 0.247.1
 Summary: A library for creating GraphQL APIs
 Home-page: https://strawberry.rocks/
 License: MIT

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/pyproject.toml

@@ -1,7 +1,7 @@
 [tool.poetry]
 name = "strawberry-graphql"
 packages = [ { include = "strawberry" } ]
-version = "0.247.0"
+version = "0.247.1"
 description = "A library for creating GraphQL APIs"
 authors = ["Patrick Arminio <patrick.arminio@gmail.com>"]
 license = "MIT"

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/strawberry/aiohttp/views.py

@@ -30,6 +30,7 @@ from strawberry.http.exceptions import (
     HTTPException,
     NonJsonMessageReceived,
     NonTextMessageReceived,
+    WebSocketDisconnected,
 )
 from strawberry.http.types import FormData, HTTPMethod, QueryParams
 from strawberry.http.typevars import (
@@ -105,7 +106,10 @@ class AioHTTPWebSocketAdapter(AsyncWebSocketAdapter):
                 raise NonTextMessageReceived()
 
     async def send_json(self, message: Mapping[str, object]) -> None:
-        await self.ws.send_json(message)
+        try:
+            await self.ws.send_json(message)
+        except RuntimeError as exc:
+            raise WebSocketDisconnected from exc
 
     async def close(self, code: int, reason: str) -> None:
         await self.ws.close(code=code, message=reason.encode())

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/strawberry/asgi/__init__.py

@@ -37,6 +37,7 @@ from strawberry.http.exceptions import (
     HTTPException,
     NonJsonMessageReceived,
     NonTextMessageReceived,
+    WebSocketDisconnected,
 )
 from strawberry.http.types import FormData, HTTPMethod, QueryParams
 from strawberry.http.typevars import (
@@ -105,7 +106,10 @@ class ASGIWebSocketAdapter(AsyncWebSocketAdapter):
             pass
 
     async def send_json(self, message: Mapping[str, object]) -> None:
-        await self.ws.send_json(message)
+        try:
+            await self.ws.send_json(message)
+        except WebSocketDisconnect as exc:
+            raise WebSocketDisconnected from exc
 
     async def close(self, code: int, reason: str) -> None:
         await self.ws.close(code=code, reason=reason)

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/strawberry/http/exceptions.py

@@ -12,4 +12,8 @@ class NonJsonMessageReceived(Exception):
     pass
 
 
+class WebSocketDisconnected(Exception):
+    pass
+
+
 __all__ = ["HTTPException"]

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/strawberry/litestar/controller.py

@@ -53,6 +53,7 @@ from strawberry.http.exceptions import (
     HTTPException,
     NonJsonMessageReceived,
     NonTextMessageReceived,
+    WebSocketDisconnected,
 )
 from strawberry.http.types import FormData, HTTPMethod, QueryParams
 from strawberry.http.typevars import Context, RootValue
@@ -216,7 +217,10 @@ class LitestarWebSocketAdapter(AsyncWebSocketAdapter):
             pass
 
     async def send_json(self, message: Mapping[str, object]) -> None:
-        await self.ws.send_json(message)
+        try:
+            await self.ws.send_json(message)
+        except WebSocketDisconnect as exc:
+            raise WebSocketDisconnected from exc
 
     async def close(self, code: int, reason: str) -> None:
         await self.ws.close(code=code, reason=reason)

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/strawberry/subscriptions/protocols/graphql_transport_ws/handlers.py

@@ -7,7 +7,6 @@ from typing import (
     TYPE_CHECKING,
     Any,
     Awaitable,
-    Callable,
     Dict,
     List,
     Optional,
@@ -15,7 +14,11 @@ from typing import (
 
 from graphql import GraphQLError, GraphQLSyntaxError, parse
 
-from strawberry.http.exceptions import NonJsonMessageReceived, NonTextMessageReceived
+from strawberry.http.exceptions import (
+    NonJsonMessageReceived,
+    NonTextMessageReceived,
+    WebSocketDisconnected,
+)
 from strawberry.subscriptions.protocols.graphql_transport_ws.types import (
     CompleteMessage,
     ConnectionAckMessage,
@@ -76,12 +79,17 @@ class BaseGraphQLTransportWSHandler:
         self.on_request_accepted()
 
         try:
-            async for message in self.websocket.iter_json():
-                await self.handle_message(message)
-        except NonTextMessageReceived:
-            await self.handle_invalid_message("WebSocket message type must be text")
-        except NonJsonMessageReceived:
-            await self.handle_invalid_message("WebSocket message must be valid JSON")
+            try:
+                async for message in self.websocket.iter_json():
+                    await self.handle_message(message)
+            except NonTextMessageReceived:
+                await self.handle_invalid_message("WebSocket message type must be text")
+            except NonJsonMessageReceived:
+                await self.handle_invalid_message(
+                    "WebSocket message must be valid JSON"
+                )
+        except WebSocketDisconnected:
+            pass
         finally:
             await self.shutdown()
 
@@ -127,50 +135,41 @@ class BaseGraphQLTransportWSHandler:
         self.task_logger.exception("Exception in worker task", exc_info=error)
 
     async def handle_message(self, message: dict) -> None:
-        handler: Callable
-        handler_arg: Any
         try:
             message_type = message.pop("type")
 
             if message_type == ConnectionInitMessage.type:
-                handler = self.handle_connection_init
-                handler_arg = ConnectionInitMessage(**message)
+                await self.handle_connection_init(ConnectionInitMessage(**message))
 
             elif message_type == PingMessage.type:
-                handler = self.handle_ping
-                handler_arg = PingMessage(**message)
+                await self.handle_ping(PingMessage(**message))
 
             elif message_type == PongMessage.type:
-                handler = self.handle_pong
-                handler_arg = PongMessage(**message)
+                await self.handle_pong(PongMessage(**message))
 
             elif message_type == SubscribeMessage.type:
-                handler = self.handle_subscribe
-
                 payload_args = message.pop("payload")
-
                 payload = SubscribeMessagePayload(
                     query=payload_args["query"],
                     operationName=payload_args.get("operationName"),
                     variables=payload_args.get("variables"),
                     extensions=payload_args.get("extensions"),
                 )
-                handler_arg = SubscribeMessage(payload=payload, **message)
+                await self.handle_subscribe(
+                    SubscribeMessage(payload=payload, **message)
+                )
 
             elif message_type == CompleteMessage.type:
-                handler = self.handle_complete
-                handler_arg = CompleteMessage(**message)
+                await self.handle_complete(CompleteMessage(**message))
 
             else:
-                handler = self.handle_invalid_message
-                handler_arg = f"Unknown message type: {message_type}"
+                error_message = f"Unknown message type: {message_type}"
+                await self.handle_invalid_message(error_message)
 
         except (KeyError, TypeError):
-            handler = self.handle_invalid_message
-            handler_arg = "Failed to parse message"
-
-        await handler(handler_arg)
-        await self.reap_completed_tasks()
+            await self.handle_invalid_message("Failed to parse message")
+        finally:
+            await self.reap_completed_tasks()
 
     async def handle_connection_init(self, message: ConnectionInitMessage) -> None:
         if self.connection_timed_out:

{strawberry_graphql-0.247.0 → strawberry_graphql-0.247.1}/strawberry/subscriptions/protocols/graphql_ws/handlers.py

@@ -11,7 +11,7 @@ from typing import (
     cast,
 )
 
-from strawberry.http.exceptions import NonTextMessageReceived
+from strawberry.http.exceptions import NonTextMessageReceived, WebSocketDisconnected
 from strawberry.subscriptions.protocols.graphql_ws import (
     GQL_COMPLETE,
     GQL_CONNECTION_ACK,
@@ -65,12 +65,17 @@ class BaseGraphQLWSHandler:
 
     async def handle(self) -> None:
         try:
-            async for message in self.websocket.iter_json(ignore_parsing_errors=True):
-                await self.handle_message(cast(OperationMessage, message))
-        except NonTextMessageReceived:
-            await self.websocket.close(
-                code=1002, reason="WebSocket message type must be text"
-            )
+            try:
+                async for message in self.websocket.iter_json(
+                    ignore_parsing_errors=True
+                ):
+                    await self.handle_message(cast(OperationMessage, message))
+            except NonTextMessageReceived:
+                await self.websocket.close(
+                    code=1002, reason="WebSocket message type must be text"
+                )
+        except WebSocketDisconnected:
+            pass
         finally:
             if self.keep_alive_task:
                 self.keep_alive_task.cancel()