Merge pull request #30 from ccnmtl/init-sentry

Refactored common sentry setup

CHANGES.txt

@@ -1,3 +1,7 @@
+0.3.4
+====================
+* Introduced init_sentry function for staging and production.
+
 0.3.3 (2024-08-15)
 ====================
 * Added token process to get_ec2_instance_ip()

ctlsettings/production.py

@@ -14,7 +14,6 @@ def common(**kwargs):
     s3static = kwargs.get('s3static', True)
     cloudfront = kwargs.get('cloudfront', None)
     s3prefix = kwargs.get('s3prefix', 'ctl')
-    sentry_dsn = kwargs.get('sentry_dsn', None)
 
     DEBUG = False
 
@@ -30,14 +29,6 @@ def common(**kwargs):
         }
     }
 
-    if sentry_dsn and \
-        ('migrate' not in sys.argv) and \
-            ('collectstatic' not in sys.argv):
-        sentry_sdk.init(
-            dsn=sentry_dsn,  # noqa: F405
-            integrations=[DjangoIntegration()],
-        )
-
     MEDIA_ROOT = '/var/www/' + project + '/uploads/'
 
     STATICMEDIA_MOUNTS = [
@@ -98,3 +89,13 @@ def common(**kwargs):
     }
 
     return locals()
+
+
+def init_sentry(sentry_dsn: str) -> None:
+    if sentry_dsn and \
+        ('migrate' not in sys.argv) and \
+            ('collectstatic' not in sys.argv):
+        sentry_sdk.init(
+            dsn=sentry_dsn,
+            integrations=[DjangoIntegration()],
+        )

ctlsettings/staging.py

@@ -14,7 +14,6 @@ def common(**kwargs):
     s3static = kwargs.get('s3static', True)
     cloudfront = kwargs.get('cloudfront', None)
     s3prefix = kwargs.get('s3prefix', 'ctl')
-    sentry_dsn = kwargs.get('sentry_dsn', None)
 
     DEBUG = False
     STAGING_ENV = True
@@ -31,15 +30,6 @@ def common(**kwargs):
         }
     }
 
-    if sentry_dsn and \
-        ('migrate' not in sys.argv) and \
-            ('collectstatic' not in sys.argv):
-        sentry_sdk.init(
-            dsn=sentry_dsn,  # noqa: F405
-            integrations=[DjangoIntegration()],
-            debug=True,
-        )
-
     STATSD_PREFIX = project + "-staging"
 
     MEDIA_ROOT = '/var/www/' + project + '/uploads/'
@@ -103,3 +93,14 @@ def common(**kwargs):
     }
 
     return locals()
+
+
+def init_sentry(sentry_dsn: str) -> None:
+    if sentry_dsn and \
+        ('migrate' not in sys.argv) and \
+            ('collectstatic' not in sys.argv):
+        sentry_sdk.init(
+            dsn=sentry_dsn,
+            integrations=[DjangoIntegration()],
+            debug=True,
+        )