lint

homewayio · Jan 9, 2025 · 05ea5a8 · 05ea5a8
1 parent 1209f53
commit 05ea5a8
Showing 1 changed file with 26 additions and 25 deletions.
diff --git a/homeway/homeway/sentry.py b/homeway/homeway/sentry.py
@@ -2,9 +2,9 @@
 import time
 import traceback
 
-import sentry_sdk
-from sentry_sdk.integrations.logging import LoggingIntegration
-from sentry_sdk.integrations.threading import ThreadingIntegration
+# import sentry_sdk
+# from sentry_sdk.integrations.logging import LoggingIntegration
+# from sentry_sdk.integrations.threading import ThreadingIntegration
 
 from .exceptions import NoSentryReportException
 
@@ -72,7 +72,8 @@ def Setup(versionString:str, distType:str, isDevMode:bool = False):
 
     @staticmethod
     def SetPluginId(pluginId:str):
-        sentry_sdk.set_context("homeway", { "plugin-id": pluginId })
+        #sentry_sdk.set_context("homeway", { "plugin-id": pluginId })
+        pass
 
 
     @staticmethod
@@ -102,14 +103,14 @@ def LogError(msg:str, extras:dict = None) -> None:
         if Sentry._Logger is None:
             return
         Sentry._Logger.error(f"Sentry Error: {msg}")
-        # Never send in dev mode, as Sentry will not be setup.
-        if Sentry.IsSentrySetup and Sentry.IsDevMode is False:
-            with sentry_sdk.push_scope() as scope:
-                scope.set_level("error")
-                if extras is not None:
-                    for key, value in extras.items():
-                        scope.set_extra(key, value)
-                sentry_sdk.capture_message(msg)
+        # # Never send in dev mode, as Sentry will not be setup.
+        # if Sentry.IsSentrySetup and Sentry.IsDevMode is False:
+        #     with sentry_sdk.push_scope() as scope:
+        #         scope.set_level("error")
+        #         if extras is not None:
+        #             for key, value in extras.items():
+        #                 scope.set_extra(key, value)
+        #         sentry_sdk.capture_message(msg)
 
 
     # Logs and reports an exception.
@@ -138,16 +139,16 @@ def _handleException(msg:str, exception:Exception, sendException:bool, extras:di
             exceptionClassType = exception.__class__.__name__
         Sentry._Logger.error(msg + "; "+str(exceptionClassType)+" Exception: " + str(exception) + "; "+str(tb))
 
-        # We have a special exception that we can throw but we won't report it to sentry.
-        # See the class for details.
-        if isinstance(exception, NoSentryReportException):
-            return
-
-        # Never send in dev mode, as Sentry will not be setup.
-        if Sentry.IsSentrySetup and sendException and Sentry.IsDevMode is False:
-            with sentry_sdk.push_scope() as scope:
-                scope.set_extra("Exception Message", msg)
-                if extras is not None:
-                    for key, value in extras.items():
-                        scope.set_extra(key, value)
-                sentry_sdk.capture_exception(exception)
+        # # We have a special exception that we can throw but we won't report it to sentry.
+        # # See the class for details.
+        # if isinstance(exception, NoSentryReportException):
+        #     return
+
+        # # Never send in dev mode, as Sentry will not be setup.
+        # if Sentry.IsSentrySetup and sendException and Sentry.IsDevMode is False:
+        #     with sentry_sdk.push_scope() as scope:
+        #         scope.set_extra("Exception Message", msg)
+        #         if extras is not None:
+        #             for key, value in extras.items():
+        #                 scope.set_extra(key, value)
+        #         sentry_sdk.capture_exception(exception)