dynamicforms-fastapi-viewsets 0.5.5__tar.gz → 0.5.7__tar.gz

{dynamicforms_fastapi_viewsets-0.5.5 → dynamicforms_fastapi_viewsets-0.5.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: dynamicforms-fastapi-viewsets
-Version: 0.5.5
+Version: 0.5.7
 Summary: Django REST Framework-style viewsets for FastAPI, with optional Celery-backed async execution and a Vue/TypeScript client counterpart.
 Project-URL: Homepage, https://github.com/dynamicforms/fastapi-viewsets
 Project-URL: Repository, https://github.com/dynamicforms/fastapi-viewsets

dynamicforms_fastapi_viewsets-0.5.7/fastapi_viewsets/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.5.7"

{dynamicforms_fastapi_viewsets-0.5.5 → dynamicforms_fastapi_viewsets-0.5.7}/fastapi_viewsets/decorators/celery_viewset/server.py

@@ -175,7 +175,12 @@ def celery_viewset_server(
                 continue
             seen_tasks.add(task_name)
 
-            celery_app.task(name=task_name)(get_sync_wrapper(route.endpoint, task_name))
+            # ignore_result=True - celery_viewset_client never reads Celery's own result backend,
+            # it awaits the redis_client.rpush() queue above instead (see client.py). Without this,
+            # Celery still calls task.backend.mark_as_done(retval=result) on every task, and a raw
+            # Pydantic model (or list of them) returned by the endpoint isn't JSON-serializable by
+            # Celery's own encoder, crashing that unused store.
+            celery_app.task(name=task_name, ignore_result=True)(get_sync_wrapper(route.endpoint, task_name))
 
         cls.__celery_viewset_metadata__ = {"task_prefix": task_prefix, "lifecycle": lifecycle, "celery_app": celery_app}
 

{dynamicforms_fastapi_viewsets-0.5.5 → dynamicforms_fastapi_viewsets-0.5.7}/fastapi_viewsets/middleware/auth/authorization.py

@@ -23,14 +23,18 @@ class Authorization(Middleware):
     - If it's callable, it's evaluated as `check(request, cls, context)` (sync or async) in
       `depends()` - bridged onto FastAPI's own `Depends()` by route_viewset - a falsy result
       rejects the request with 403 before `perform_*` (or even FastAPI's own body parsing) ever
-      runs. Use this for checks that don't need a specific record (e.g. a role check).
-    - Either way (callable or not, or even unconfigured), the resolved value is made available as
-      `context.authorization` (set in `__call__`, unconditionally, in the onion chain - there's no
-      reject decision here, so there's no benefit to doing this any earlier) - `perform_*` can
-      inspect it once it has whatever record-specific info a check needs (e.g. the object a `pk`
-      resolved to) and reject itself (`raise HTTPException(403, ...)`) if it decides to. This is
-      the "no hook in the middleware layer needed" case - just raise, same as anywhere else in the
-      pipeline (see docs/guide/action-configuration.md's "Rejecting a request: just raise").
+      runs. Use this for checks that don't need a specific record (e.g. a role check). A callable
+      config is consumed entirely by `depends()` and never reaches `context.authorization` - there
+      is nothing left for `perform_*` to do with the check function itself, and a bare function
+      isn't something a `celery_viewset` worker could receive anyway.
+    - Otherwise (non-callable, or unconfigured/`None`), nothing is enforced by the middleware
+      itself; the value is made available as `context.authorization` (set in `__call__`, in the
+      onion chain - there's no reject decision here, so there's no benefit to doing this any
+      earlier) - `perform_*` can inspect it once it has whatever record-specific info a check
+      needs (e.g. the object a `pk` resolved to) and reject itself (`raise HTTPException(403,
+      ...)`) if it decides to. This is the "no hook in the middleware layer needed" case - just
+      raise, same as anywhere else in the pipeline (see docs/guide/action-configuration.md's
+      "Rejecting a request: just raise").
 
     Unconfigured (no `@action_configuration` entry for `Authorization` at all): nothing is
     enforced, `context.authorization` resolves to `None`.
@@ -63,5 +67,7 @@ class Authorization(Middleware):
         context: "Context",
         call_next: Callable[[], Awaitable[ViewSetResult]],
     ) -> ViewSetResult:
-        context.set("authorization", self.config_from(context))
+        config = self.config_from(context)
+        if not callable(config):
+            context.set("authorization", config)
         return await call_next()

{dynamicforms_fastapi_viewsets-0.5.5 → dynamicforms_fastapi_viewsets-0.5.7}/fastapi_viewsets/mux_ws/protocol.py

@@ -37,8 +37,6 @@ consume the whole body before learning whether it was an error at all.
 from typing import Any
 from urllib.parse import urlencode
 
-from muxws.frames import ABSENT
-
 METHOD_KEY = ":method"
 PATH_KEY = ":path"
 QUERY_KEY = ":query"
@@ -120,4 +118,6 @@ def build_response_meta(status: int, headers: dict[str, str]) -> dict[str, Any]:
 
 def body_to_payload(body: Any) -> Any:
     """`ABSENT` is muxws' "no payload at all", which is a different frame from a null payload."""
+    from muxws.frames import ABSENT
+
     return ABSENT if body is None else body

dynamicforms_fastapi_viewsets-0.5.5/fastapi_viewsets/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.5.5"