refactor: define response outside try-except

openedx · Oct 15, 2024 · 6ebab7c · 6ebab7c
1 parent a119e81
commit 6ebab7c
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/event_routing_backends/utils/xapi_lrs_client.py b/event_routing_backends/utils/xapi_lrs_client.py
@@ -72,11 +72,14 @@ def bulk_send(self, statement_data):
             requests.Response object
         """
         logger.debug('Sending {} xAPI statements to {}'.format(len(statement_data), self.URL))
+        response = None
 
         try:
             response = self.lrs_client.save_statements(statement_data)
-        except JSONDecodeError as e:
+        except JSONDecodeError:
             logger.warning(f"Events already in LRS: {response.request.content}")
+
+        if not response:
             return
 
         if not response.success: