fix: Remove "/" from crash report file name

CHANGELOG.md

@@ -29,6 +29,7 @@ This change has no impact on grouping of the issues in Sentry.
 ### Fixes 
 
 - Propagate span when copying scope (#2952)
+- Remove "/" from crash report file name (#3005)
 
 ## 8.6.0
 

Sources/SentryCrash/Recording/SentryCrash.m

@@ -67,10 +67,19 @@
 static NSString *
 getBundleName(void)
 {
-    NSString *bundleName = [[[NSBundle mainBundle] infoDictionary] objectForKey:@"CFBundleName"];
+    static NSString *bundleName = nil;
+
     if (bundleName == nil) {
-        bundleName = @"Unknown";
+#if TEST
+        bundleName = @"Sentry/Test";
+#else
+        bundleName =
+            [[[NSBundle mainBundle] infoDictionary] objectForKey:@"CFBundleName"] ?: @"Unknown";
+#endif
+        // bundleName is only used for file name, therefore '/' is not allowed.
+        bundleName = [bundleName stringByReplacingOccurrencesOfString:@"/" withString:@"-"];
     }
+
     return bundleName;
 }
 

Tests/SentryTests/SentryCrash/SentryCrashTests.m

@@ -7,7 +7,11 @@ @interface SentryCrashTests : FileBasedTestCase
 @end
 
 @interface SentryCrash (private)
+
+@property (nonatomic, readwrite, retain) NSString *bundleName;
+
 - (NSArray *)getAttachmentPaths:(int64_t)reportID;
+
 @end
 
 @implementation SentryCrashTests
@@ -43,6 +47,12 @@ - (void)test_getScreenshots_TwoFiles
     XCTAssertEqual(files.count, 2);
 }
 
+- (void)test_cleanBundleName
+{
+    SentryCrash *sentryCrash = [[SentryCrash alloc] init];
+    XCTAssertEqualObjects(sentryCrash.bundleName, @"Sentry-Test");
+}
+
 - (void)test_getScreenshots_NoFiles
 {
     [self initReport:12 withScreenshots:0];