fix(replay): MaskAll* set to false should unmask all

getsentry · Nov 8, 2024 · 36f4d74 · 36f4d74
1 parent 0b19589
commit 36f4d74
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 4 deletions.
diff --git a/packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryReplayOptionsTests.swift b/packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryReplayOptionsTests.swift
@@ -67,8 +67,8 @@ final class RNSentryReplayOptions: XCTestCase {
         XCTAssertEqual(replayOptions.count, 5)
         XCTAssertNotNil(replayOptions["sessionSampleRate"])
         XCTAssertNotNil(replayOptions["errorSampleRate"])
-        XCTAssertNotNil(replayOptions["redactAllImages"])
-        XCTAssertNotNil(replayOptions["redactAllText"])
+        XCTAssertNotNil(replayOptions["maskAllImages"])
+        XCTAssertNotNil(replayOptions["maskAllText"])
         XCTAssertNotNil(replayOptions["maskedViewClasses"])
     }
 
@@ -133,6 +133,21 @@ final class RNSentryReplayOptions: XCTestCase {
         XCTAssertEqual(ObjectIdentifier(actualOptions.experimental.sessionReplay.maskedViewClasses[0]), ObjectIdentifier(NSClassFromString("RCTImageView")!))
     }
 
+    func testMaskAllImagesFalse() {
+        let optionsDict = ([
+            "dsn": "https://[email protected]/1234567",
+            "_experiments": [ "replaysOnErrorSampleRate": 0.75 ],
+            "mobileReplayOptions": [ "maskAllImages": false ]
+        ] as NSDictionary).mutableCopy() as! NSMutableDictionary
+
+        RNSentryReplay.updateOptions(optionsDict)
+
+        let actualOptions = try! Options(dict: optionsDict as! [String: Any])
+
+        XCTAssertEqual(actualOptions.experimental.sessionReplay.maskAllImages, false)
+        XCTAssertEqual(actualOptions.experimental.sessionReplay.maskedViewClasses.count, 0)
+    }
+
     func testMaskAllText() {
         let optionsDict = ([
             "dsn": "https://[email protected]/1234567",
@@ -152,4 +167,19 @@ final class RNSentryReplayOptions: XCTestCase {
         XCTAssertEqual(ObjectIdentifier(actualOptions.experimental.sessionReplay.maskedViewClasses[1]), ObjectIdentifier(NSClassFromString("RCTParagraphComponentView")!))
     }
 
+    func testMaskAllTextFalse() {
+        let optionsDict = ([
+            "dsn": "https://[email protected]/1234567",
+            "_experiments": [ "replaysOnErrorSampleRate": 0.75 ],
+            "mobileReplayOptions": [ "maskAllText": false ]
+        ] as NSDictionary).mutableCopy() as! NSMutableDictionary
+
+        RNSentryReplay.updateOptions(optionsDict)
+
+        let actualOptions = try! Options(dict: optionsDict as! [String: Any])
+
+        XCTAssertEqual(actualOptions.experimental.sessionReplay.maskAllText, false)
+        XCTAssertEqual(actualOptions.experimental.sessionReplay.maskedViewClasses.count, 0)
+    }
+
 }
diff --git a/packages/core/ios/RNSentryReplay.m b/packages/core/ios/RNSentryReplay.m
@@ -28,8 +28,8 @@ + (void)updateOptions:(NSMutableDictionary *)options
         @"sessionReplay" : @ {
             @"sessionSampleRate" : experiments[@"replaysSessionSampleRate"] ?: [NSNull null],
             @"errorSampleRate" : experiments[@"replaysOnErrorSampleRate"] ?: [NSNull null],
-            @"redactAllImages" : replayOptions[@"maskAllImages"] ?: [NSNull null],
-            @"redactAllText" : replayOptions[@"maskAllText"] ?: [NSNull null],
+            @"maskAllImages" : replayOptions[@"maskAllImages"] ?: [NSNull null],
+            @"maskAllText" : replayOptions[@"maskAllText"] ?: [NSNull null],
             @"maskedViewClasses" : [RNSentryReplay getReplayRNRedactClasses:replayOptions],
         }
     }