Bugfix: CRUDEvent signal error log try/except (#133)

soynatan · Apr 1, 2020 · 3cdc719 · 3cdc719
1 parent d772175
commit 3cdc719
Showing 1 changed file with 24 additions and 12 deletions.
diff --git a/easyaudit/signals/model_signals.py b/easyaudit/signals/model_signals.py
@@ -112,9 +112,12 @@ def crud_flow():
                                 'user_pk_as_string': str(user.pk) if user else user
                             })
                     except Exception as e:
-                        logger.exception(
-                            "easy audit had a pre-save exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
-                                instance, instance.pk))
+                        try:
+                            logger.exception(
+                                "easy audit had a pre_save exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
+                                    instance, instance.pk))
+                        except Exception:
+                            pass
                 if getattr(settings, "TEST", False):
                     crud_flow()
                 else:
@@ -175,9 +178,12 @@ def crud_flow():
                                 'user_pk_as_string': str(user.pk) if user else user
                             })
                     except Exception as e:
-                        logger.exception(
-                            "easy audit had a pre-save exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
-                                instance, instance.pk))
+                        try:
+                            logger.exception(
+                                "easy audit had a post_save exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
+                                    instance, instance.pk))
+                        except Exception:
+                            pass
                 if getattr(settings, "TEST", False):
                     crud_flow()
                 else:
@@ -258,9 +264,12 @@ def crud_flow():
                             'user_pk_as_string': str(user.pk) if user else user
                         })
                 except Exception as e:
-                    logger.exception(
-                        "easy audit had a pre-save exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
-                            instance, instance.pk))
+                    try:
+                        logger.exception(
+                            "easy audit had a m2m_changed exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
+                                instance, instance.pk))
+                    except Exception:
+                        pass
 
             if getattr(settings, "TEST", False):
                 crud_flow()
@@ -307,9 +316,12 @@ def crud_flow():
                         })
 
                 except Exception as e:
-                    logger.exception(
-                        "easy audit had a pre-save exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
-                            instance, instance.pk))
+                    try:
+                        logger.exception(
+                            "easy audit had a post_delete exception on CRUDEvent creation. instance: {}, instance pk: {}".format(
+                                instance, instance.pk))
+                    except Exception:
+                        pass
 
             if getattr(settings, "TEST", False):
                 crud_flow()