finalize endpoints before dispatching async tasks (#137)

* finalize endpoints before dispatching async tasks

* changelog and bumpversion

Author: dshafer

.bumpversion.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 0.31.3
+current_version = 0.31.4
 
 [bumpversion:file:pyproject.toml]
 

CHANGELOG.md

@@ -4,6 +4,9 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+# [0.31.4] - 2023-10-12
+- [PR 137](https://github.com/salesforce/django-declarative-apis/pull/137) Finalize endpoints before dispatching async tasks
+
 # [0.31.3] - 2023-09-30
 - [PR 136](https://github.com/salesforce/django-declarative-apis/pull/136) Fix filter expansion for pydantic objects
 

django_declarative_apis/machinery/__init__.py

@@ -265,6 +265,9 @@ def get_response(self):  # noqa: C901
                 if resource and resource.is_dirty(check_relationship=True):
                     update_dirty(resource)
 
+            # all synchronous tasks are done, finalize the endpoint before launching async tasks
+            self.bound_endpoint.finalize()
+
             for deferred_task in deferred_tasks:
                 deferred_task.run(self.bound_endpoint)
 
@@ -449,7 +452,6 @@ def process_request_and_get_response(self, request, *args, **kwargs):
                 bound_endpoint.bound_endpoint.__class__.__name__,
             )
             result = bound_endpoint.get_response()
-            bound_endpoint.bound_endpoint.finalize()
             return result
         except errors.ApiError:
             raise

docs/source/conf.py

@@ -74,7 +74,7 @@
 # built documents.
 
 # The full version, including alpha/beta/rc tags.
-release = '0.31.3'  # set by bumpversion
+release = '0.31.4'  # set by bumpversion
 
 # The short X.Y version.
 version = release.rsplit('.', 1)[0]

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "django-declarative-apis"
-version = "0.31.3"  # set by bumpversion
+version = "0.31.4"  # set by bumpversion
 description = "Simple, readable, declarative APIs for Django"
 readme = "README.md"
 dependencies = [