Merge pull request #44 from ccnmtl/sentry-fix

sentry-js: Wait for onLoad before using Sentry.setUser()
ccnmtl · Oct 12, 2024 · 5706e7b · 5706e7b
2 parents e7b3a6d + 2059b77
commit 5706e7b
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 10 deletions.
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -1,3 +1,7 @@
+0.4.2
+====================
+* Updated sentry-js config to use Sentry.onLoad.
+
 0.4.1 (2024-10-11)
 ====================
 * Updated sentry JS config to use username instead of id.

diff --git a/ctlsettings/templates/ctlsettings/sentry_js.html b/ctlsettings/templates/ctlsettings/sentry_js.html
@@ -6,17 +6,21 @@
             environment: '{{ ENVIRONMENT }}'
         });
 
-        {% if request.user.is_anonymous %}
-        Sentry.setUser({
-            email: 'none',
-            username: 'anonymous'
-        });
-        {% else %}
-        Sentry.setUser({
-            email: '{{ request.user.email }}',
-            username: '{{ request.user.username }}'
+        Sentry.onLoad(function() {
+            const client = Sentry.getClient();
+
+            {% if request.user.is_anonymous %}
+            client.setUser({
+                email: 'none',
+                username: 'anonymous'
+            });
+            {% else %}
+            client.setUser({
+                email: '{{ request.user.email }}',
+                username: '{{ request.user.username }}'
+            });
+            {% endif %}
         });
-        {% endif %}
     };
 </script>