From 85e2a846429a0d02f2587ea426c0cdc449fae9d9 Mon Sep 17 00:00:00 2001
From: jpitts-uva <jp2pz@uvahealth.org>
Date: Wed, 17 Jan 2024 06:03:59 -0500
Subject: [PATCH] Fix to prevent continuing Sentry "Working outside of
 application context" error.

---
 crc/services/workflow_service.py | 45 ++++++++++++++++----------------
 1 file changed, 23 insertions(+), 22 deletions(-)

diff --git a/crc/services/workflow_service.py b/crc/services/workflow_service.py
index e8ead7ee..89eb3e9d 100755
--- a/crc/services/workflow_service.py
+++ b/crc/services/workflow_service.py
@@ -139,30 +139,31 @@ class WorkflowService(object):
         return workflow_url
 
     def process_erroring_workflows(self):
-        workflows = self.get_erroring_workflows()
-        if len(workflows) > 0:
-            workflow_urls = []
-            if len(workflows) == 1:
-                workflow = workflows[0]
-                workflow_url_link = self.get_workflow_url(workflow)
-                workflow_urls.append(workflow_url_link)
-                message = 'There is one workflow in an error state.'
-                message += f'\n You can restart the workflow at {workflow_url_link}.'
-            else:
-                message = f'There are {len(workflows)} workflows in an error state.'
-                message += '\nYou can restart the workflows at these URLs:'
-                for workflow in workflows:
+        with app.app_context():    
+            workflows = self.get_erroring_workflows()
+            if len(workflows) > 0:
+                workflow_urls = []
+                if len(workflows) == 1:
+                    workflow = workflows[0]
                     workflow_url_link = self.get_workflow_url(workflow)
                     workflow_urls.append(workflow_url_link)
-                    message += f'\n{workflow_url_link}'
-
-            with push_scope() as scope:
-                scope.user = {"urls": workflow_urls}
-                scope.set_extra("workflow_urls", workflow_urls)
-                # this sends a message through sentry
-                capture_message(message)
-            # We return message so we can use it in a test
-            return message
+                    message = 'There is one workflow in an error state.'
+                    message += f'\n You can restart the workflow at {workflow_url_link}.'
+                else:
+                    message = f'There are {len(workflows)} workflows in an error state.'
+                    message += '\nYou can restart the workflows at these URLs:'
+                    for workflow in workflows:
+                        workflow_url_link = self.get_workflow_url(workflow)
+                        workflow_urls.append(workflow_url_link)
+                        message += f'\n{workflow_url_link}'
+    
+                with push_scope() as scope:
+                    scope.user = {"urls": workflow_urls}
+                    scope.set_extra("workflow_urls", workflow_urls)
+                    # this sends a message through sentry
+                    capture_message(message)
+                # We return message so we can use it in a test
+                return message
 
     @staticmethod
     def test_spec(spec_id, validate_study_id=None, test_until=None, required_only=False):