fix: Delete old session replay files

CHANGELOG.md

@@ -19,7 +19,7 @@ via the option `swizzleClassNameExclude`.
 - Finish TTID correctly when viewWillAppear is skipped (#4417)
 - Swizzling RootUIViewController if ignored by `swizzleClassNameExclude` (#4407)
 - Data race in SentrySwizzleInfo.originalCalled (#4434)
-
+- Delete old session replay files (#4446)
 
 ### Improvements
 

Sources/Sentry/SentrySessionReplayIntegration.m

@@ -89,6 +89,8 @@
     _notificationCenter = SentryDependencyContainer.sharedInstance.notificationCenterWrapper;
 
     [self moveCurrentReplay];
+    [self cleanUp];
+
     [SentrySDK.currentHub registerSessionListener:self];
     [SentryGlobalEventProcessor.shared
         addEventProcessor:^SentryEvent *_Nullable(SentryEvent *_Nonnull event) {
@@ -103,6 +105,19 @@
     [SentryDependencyContainer.sharedInstance.reachability addObserver:self];
 }
 
+- (nullable NSDictionary<NSString *, id> *)lastReplayInfo
+{
+    NSURL *dir = [self replayDirectory];
+    NSURL *lastReplayUrl = [dir URLByAppendingPathComponent:SENTRY_LAST_REPLAY];
+    NSData *lastReplay = [NSData dataWithContentsOfURL:lastReplayUrl];
+
+    if (lastReplay == nil) {
+        return nil;
+    }
+
+    return [SentrySerialization deserializeDictionaryFromJsonData:lastReplay];
+}
+
 /**
  * Send the cached frames from a previous session that eventually crashed.
  * This function is called when processing an event created by SentryCrashIntegration,
@@ -112,15 +127,8 @@
 - (void)resumePreviousSessionReplay:(SentryEvent *)event
 {
     NSURL *dir = [self replayDirectory];
-    NSURL *lastReplayUrl = [dir URLByAppendingPathComponent:SENTRY_LAST_REPLAY];
-    NSData *lastReplay = [NSData dataWithContentsOfURL:lastReplayUrl];
+    NSDictionary<NSString *, id> *jsonObject = [self lastReplayInfo];
 
-    if (lastReplay == nil) {
-        return;
-    }
-
-    NSDictionary<NSString *, id> *jsonObject =
-        [SentrySerialization deserializeDictionaryFromJsonData:lastReplay];
     if (jsonObject == nil) {
         return;
     }
@@ -359,6 +367,43 @@
     }
 }
 
+- (void)cleanUp
+{
+    NSURL *replayDir = [self replayDirectory];
+    NSDictionary<NSString *, id> *lastReplayInfo = [self lastReplayInfo];
+    NSString *lastReplayFolder = lastReplayInfo[@"path"];
+
+    // Mapping replay folder here and not in dispatched queue to prevent a race condition between
+    // listing files and creating a new replay session.
+    NSArray *replayFiles =
+        [SentryDependencyContainer.sharedInstance.fileManager allFilesInFolder:replayDir.path];
+    if (replayFiles.count == 0) {
+        return;
+    }
+
+    [SentryDependencyContainer.sharedInstance.dispatchQueueWrapper dispatchAsyncWithBlock:^{
+        NSFileManager *fileManager = [NSFileManager defaultManager];
+        for (NSString *file in replayFiles) {
+            NSString *filePath = [replayDir.path stringByAppendingPathComponent:file];
+
+            // Skip the last replay folder.
+            if ([file isEqualToString:lastReplayFolder]) {
+                continue;
+            }
+
+            // Check if the file exists and is a directory before deleting it.
+            BOOL isDirectory = NO;
+            if ([fileManager fileExistsAtPath:filePath isDirectory:&isDirectory] && isDirectory) {
+                NSError *error = nil;
+                if (![fileManager removeItemAtPath:filePath error:&error]) {
+                    SENTRY_LOG_ERROR(@"Error deleting file at path: %@, error: %@", filePath,
+                        error.localizedDescription);
+                }
+            }
+        }
+    }];
+}
+
 - (void)pause
 {
     [self.sessionReplay pause];

Sources/Sentry/include/SentryFileManager.h

@@ -89,6 +89,7 @@ SENTRY_NO_INIT
 - (NSNumber *_Nullable)readTimezoneOffset;
 - (void)storeTimezoneOffset:(NSInteger)offset;
 - (void)deleteTimezoneOffset;
+- (NSArray<NSString *> *)allFilesInFolder:(NSString *)path;
 
 BOOL createDirectoryIfNotExists(NSString *path, NSError **error);
 

Tests/SentryTests/Integrations/SessionReplay/SentrySessionReplayIntegrationTests.swift

@@ -358,12 +358,26 @@ class SentrySessionReplayIntegrationTests: XCTestCase {
         XCTAssertTrue(sessionReplay.isFullSession)
     }
 
-    func createLastSessionReplay(writeSessionInfo: Bool = true, errorSampleRate: Double = 1) throws {
-        let options = Options()
-        options.dsn = "https://[email protected]/test"
-        options.cacheDirectoryPath = FileManager.default.temporaryDirectory.path
+    func testCleanUp() throws {
+        // Create 3 old Sessions
+        try createLastSessionReplay()
+        try createLastSessionReplay()
+        try createLastSessionReplay()
+        SentryDependencyContainer.sharedInstance().dispatchQueueWrapper = TestSentryDispatchQueueWrapper()
 
-        let replayFolder = options.cacheDirectoryPath + "/io.sentry/\(options.parsedDsn?.getHash() ?? "")/replay"
+        // Start the integration with a configuration that will enable it
+        startSDK(sessionSampleRate: 0, errorSampleRate: 1)
+
+        // Check whether there is only one old session directory and the current session directory
+        let content = try FileManager.default.contentsOfDirectory(atPath: replayFolder()).filter { name in
+            !name.hasPrefix("replay") && !name.hasPrefix(".") //remove replay info files and system directories
+        }
+
+        XCTAssertEqual(content.count, 2)
+    }
+
+    func createLastSessionReplay(writeSessionInfo: Bool = true, errorSampleRate: Double = 1) throws {
+        let replayFolder = replayFolder()
         let jsonPath = replayFolder + "/replay.current"
         var sessionFolder = UUID().uuidString
         let info: [String: Any] = ["replayId": SentryId().sentryIdString,
@@ -389,6 +403,13 @@ class SentrySessionReplayIntegrationTests: XCTestCase {
             sentrySessionReplaySync_writeInfo()
         }
     }
+
+    func replayFolder() -> String {
+        let options = Options()
+        options.dsn = "https://[email protected]/test"
+        options.cacheDirectoryPath = FileManager.default.temporaryDirectory.path
+        return options.cacheDirectoryPath + "/io.sentry/\(options.parsedDsn?.getHash() ?? "")/replay"
+    }
 }
 
 #endif