Sentry: Fix broken configureScope() call

`configureScope()` has apparently been deprecated a while ago and was recently removed in getsentry/sentry-javascript#9887. Since we are mocking Sentry in our test suite we unfortunately didn't notice the issue before it popped up on our staging environment.
Turbo87 · May 14, 2024 · 4266cc9 · 4266cc9
1 parent 271af2e
commit 4266cc9
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/app/routes/application.js b/app/routes/application.js
@@ -77,7 +77,7 @@ export default class ApplicationRoute extends Route {
   setSentryTransaction(transition) {
     let name = transition.to?.name;
     if (name) {
-      this.sentry.configureScope(scope => scope.setTransactionName(name));
+      this.sentry.getCurrentScope().setTransactionName(name);
     }
   }
 }
diff --git a/app/services/sentry.js b/app/services/sentry.js
@@ -7,8 +7,8 @@ export default class SentryService extends Service {
     Sentry.captureException(error, captureContext);
   }
 
-  configureScope(callback) {
-    Sentry.configureScope(callback);
+  getCurrentScope() {
+    return Sentry.getCurrentScope();
   }
 
   setUser(user) {

diff --git a/tests/helpers/sentry.js b/tests/helpers/sentry.js
@@ -11,8 +11,8 @@ class MockSentryService extends Service {
     this.events.push(event);
   }
 
-  configureScope(callback) {
-    callback(this.scope);
+  getCurrentScope() {
+    return this.scope;
   }
 
   setUser(user) {