Merge pull request #126 from edly-io/ERTE-101

[ERTE-101] fix: separated logging for NoTransformerImplemented and NoBackendEnabled
openedx · Oct 26, 2021 · 269486f · 269486f
2 parents b3f5e00 + cce82ac
commit 269486f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/eventtracking/backends/routing.py b/eventtracking/backends/routing.py
@@ -135,12 +135,14 @@ def send_to_backends(self, event):
         for name, backend in self.backends.items():
             try:
                 backend.send(event)
-            except (NoTransformerImplemented, NoBackendEnabled) as exc:
-                LOG.info(
-                    '[send_to_backends] Failed to send event [%s] with backend [%s], [%s]',
-                    event, backend, repr(exc)
-                )
+            except NoTransformerImplemented as exc:
+                LOG.info('[send_to_backends] No transformer has been implemented for edx event "%s", [%s]',
+                         event["name"], repr(exc))
+            except NoBackendEnabled as exc:
+                LOG.info('[send_to_backends] Failed to send edx event "%s" to "%s" backend. "%s" backend has'
+                         ' not been enabled, [%s]', event["name"], name, name, repr(exc)
+                         )
             except Exception:  # pylint: disable=broad-except
                 LOG.exception(
-                    'Unable to send event to backend: %s', name
+                    'Unable to send edx event "%s" to backend: %s', event["name"], name
                 )
diff --git a/setup.py b/setup.py
@@ -49,7 +49,7 @@ def load_requirements(*requirements_paths):
 
 setup(
     name='event-tracking',
-    version='1.1.2',
+    version='1.1.3',
     packages=find_packages(),
     include_package_data=True,
     license='AGPLv3 License',