Merge pull request #64 from labthings/fix-task-request-context

Fix task request context

Author: jtc42

labthings/core/tasks/thread.py

@@ -36,6 +36,13 @@ def __init__(self, target, *args, **kwargs):
         # Nice string representation of target function
         self.target_string = f"{self._target}(args={self._args}, kwargs={self._kwargs})"
 
+        # copy_current_request_context allows threads to access flask current_app
+        if has_request_context():
+            logging.debug(f"Copying request context to {self._target}")
+            self._target = copy_current_request_context(self._target)
+        else:
+            logging.debug("No request context to copy")
+
         # Private state properties
         self._status: str = "idle"  # Task status
         self._return_value = None  # Return value
@@ -79,12 +86,7 @@ def update_data(self, data: dict):
         self.data.update(data)
 
     def _run(self):  # pylint: disable=E0202
-        # copy_current_request_context allows threads to access flask current_app
-        if has_request_context():
-            target = copy_current_request_context(self._target)
-        else:
-            target = self._target
-        return self._thread_proc(target)(*self._args, **self._kwargs)
+        return self._thread_proc(self._target)(*self._args, **self._kwargs)
 
     def _thread_proc(self, f):
         """