replace ENABLE_SENTRY by SENTRY_DSN

.env.example

@@ -1,8 +1,8 @@
 # Server url
 URL=http://localhost:7777
 
-# Enable sentry
-ENABLE_SENTRY=true
+# If you want to enable sentry for error tracking, put your sentry dsn here.
+# SENTRY_DSN=your_sentry_dsn
 
 # apply colored log
 COLOR_LOG=true

app/config.py

@@ -34,8 +34,7 @@ PROMO_CODE = "SIMPLEISBETTER"
 URL = os.environ["URL"]
 print(">>> URL:", URL)
 
-# Whether sentry is enabled
-ENABLE_SENTRY = "ENABLE_SENTRY" in os.environ
+SENTRY_DSN = os.environ.get("SENTRY_DSN")
 
 # Email related settings
 NOT_SEND_EMAIL = "NOT_SEND_EMAIL" in os.environ

server.py

@@ -17,7 +17,7 @@ from app.auth.base import auth_bp
 from app.config import (
     DB_URI,
     FLASK_SECRET,
-    ENABLE_SENTRY,
+    SENTRY_DSN,
     URL,
     SHA1,
     PADDLE_MONTHLY_PRODUCT_ID,
@@ -42,10 +42,10 @@ from app.models import (
 from app.monitor.base import monitor_bp
 from app.oauth.base import oauth_bp
 
-if ENABLE_SENTRY:
+if SENTRY_DSN:
     LOG.d("enable sentry")
     sentry_sdk.init(
-        dsn="https://ad2187ed843340a1b4165bd8d5d6cdce@sentry.io/1478143",
+        dsn=SENTRY_DSN,
         integrations=[FlaskIntegration()],
     )
 
@@ -260,7 +260,7 @@ def jinja2_filter(app):
     @app.context_processor
     def inject_stage_and_region():
         return dict(
-            YEAR=arrow.now().year, URL=URL, ENABLE_SENTRY=ENABLE_SENTRY, VERSION=SHA1
+            YEAR=arrow.now().year, URL=URL, SENTRY_DSN=SENTRY_DSN, VERSION=SHA1
         )
 
 

templates/base.html

@@ -92,10 +92,10 @@
 </div>
 
 <script>
-  {% if ENABLE_SENTRY %}
+  {% if SENTRY_DSN %}
     console.log("Init sentry");
     try {
-      Sentry.init({dsn: 'https://ad2187ed843340a1b4165bd8d5d6cdce@sentry.io/1478143'});
+      Sentry.init({dsn: '{{SENTRY_DSN}}'});
     } catch (error) {
       console.log("Sentry error, probably due to AdBlocker ...")
     }

tests/env.test

@@ -1,9 +1,6 @@
 # Server url
 URL=http://localhost
 
-# Enable sentry
-# ENABLE_SENTRY=true
-
 # Email related settings
 # Only print email content, not sending it
 NOT_SEND_EMAIL=true