feat: stop profiler when app moves to background (#2331)

* feat: stop profiler when app moves to background
getsentry · Oct 27, 2022 · d47c2c0 · d47c2c0
1 parent 7d3d1a3
commit d47c2c0
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@
 
 - Profile concurrent transactions (#2227)
 
+### Fixes
+
+- Stop profiler when app moves to background (#2331)
+
 ## 7.29.0
 
 ### Features

diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
@@ -158,6 +158,12 @@ + (void)startForSpanID:(SentrySpanId *)spanID
                                            selector:@selector(timeoutAbort)
                                            userInfo:nil
                                             repeats:NO];
+#        if SENTRY_HAS_UIKIT
+        [[NSNotificationCenter defaultCenter] addObserver:self
+                                                 selector:@selector(backgroundAbort)
+                                                     name:UIApplicationWillResignActiveNotification
+                                                   object:nil];
+#        endif // SENTRY_HAS_UIKIT
         _gCurrentProfiler->_hub = hub;
     }
 
@@ -259,6 +265,19 @@ + (void)timeoutAbort
     [self stopProfilerForReason:SentryProfilerTruncationReasonTimeout];
 }
 
++ (void)backgroundAbort
+{
+    std::lock_guard<std::mutex> l(_gProfilerLock);
+
+    if (_gCurrentProfiler == nil) {
+        SENTRY_LOG_DEBUG(@"No current profiler to stop.");
+        return;
+    }
+
+    SENTRY_LOG_DEBUG(@"Stopping profiler %@ due to timeout.", _gCurrentProfiler);
+    [self stopProfilerForReason:SentryProfilerTruncationReasonAppMovedToBackground];
+}
+
 + (void)stopProfilerForReason:(SentryProfilerTruncationReason)reason
 {
     [_gCurrentProfiler->_timeoutTimer invalidate];