Add report view hierarchy identifier option

Author: buenaflor

flutter/lib/src/sentry_flutter_options.dart

@@ -213,6 +213,10 @@ class SentryFlutterOptions extends SentryOptions {
   @experimental
   bool attachViewHierarchy = false;
 
+  /// Enables reporting information of identifiers of the view hierarchy.
+  /// This might contain sensitive information.
+  bool reportViewHierarchyIdentifiers = true;
+
   /// When enabled, the SDK tracks when the application stops responding for a
   /// specific amount of time, See [appHangTimeoutInterval].
   /// Only available on iOS and macOS.

flutter/lib/src/view_hierarchy/sentry_tree_walker.dart

@@ -209,9 +209,10 @@ import '../widget_utils.dart';
 class _TreeWalker {
   static const _privateDelimiter = '_';
 
-  _TreeWalker(this.rootElement);
+  _TreeWalker(this.rootElement, this.options);
 
   final Element rootElement;
+  final SentryFlutterOptions options;
 
   ValueChanged<Element> _visitor(
       SentryViewHierarchyElement parentSentryElement) {
@@ -278,10 +279,15 @@ class _TreeWalker {
       alpha = widget.opacity;
     }
 
+    String? identifier;
+    if (options.reportViewHierarchyIdentifiers) {
+      identifier = WidgetUtils.toStringValue(widget.key);
+    }
+
     return SentryViewHierarchyElement(
       element.widget.runtimeType.toString(),
       depth: element.depth,
-      identifier: WidgetUtils.toStringValue(element.widget.key),
+      identifier: identifier,
       width: width,
       height: height,
       x: x,
@@ -292,15 +298,16 @@ class _TreeWalker {
   }
 }
 
-SentryViewHierarchy? walkWidgetTree(WidgetsBinding instance) {
+SentryViewHierarchy? walkWidgetTree(
+    WidgetsBinding instance, SentryFlutterOptions options) {
   // to keep compatibility with older versions
   // ignore: deprecated_member_use
   final rootElement = instance.renderViewElement;
   if (rootElement == null) {
     return null;
   }
 
-  final walker = _TreeWalker(rootElement);
+  final walker = _TreeWalker(rootElement, options);
 
   return walker.toSentryViewHierarchy();
 }

flutter/lib/src/view_hierarchy/view_hierarchy_event_processor.dart

@@ -23,7 +23,7 @@ class SentryViewHierarchyEventProcessor implements EventProcessor {
     if (instance == null) {
       return event;
     }
-    final sentryViewHierarchy = walkWidgetTree(instance);
+    final sentryViewHierarchy = walkWidgetTree(instance, _options);
 
     if (sentryViewHierarchy == null) {
       return event;

flutter/test/view_hierarchy/view_hierarchy_event_processor_test.dart

@@ -1,3 +1,5 @@
+import 'dart:convert';
+
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:sentry/sentry.dart';
@@ -79,6 +81,27 @@ void main() {
         expect(hint.viewHierarchy, isNull);
       });
     });
+
+    testWidgets('does not add view hierarchy identifiers if opt out in options',
+        (tester) async {
+      await tester.runAsync(() async {
+        final sut =
+            fixture.getSut(instance, reportViewHierarchyIdentifiers: false);
+
+        await tester.pumpWidget(MyApp());
+
+        final event = SentryEvent(
+            exceptions: [SentryException(type: 'type', value: 'value')]);
+        final hint = Hint();
+
+        sut.apply(event, hint);
+
+        expect(hint.viewHierarchy, isNotNull);
+        final bytes = await hint.viewHierarchy!.bytes;
+        final jsonString = utf8.decode(bytes);
+        expect(jsonString, isNot(contains('identifier')));
+      });
+    });
   });
 }
 
@@ -99,9 +122,11 @@ class TestBindingWrapper implements BindingWrapper {
 }
 
 class Fixture {
-  SentryViewHierarchyEventProcessor getSut(WidgetsBinding instance) {
+  SentryViewHierarchyEventProcessor getSut(WidgetsBinding instance,
+      {bool reportViewHierarchyIdentifiers = true}) {
     final options = SentryFlutterOptions()
-      ..bindingUtils = TestBindingWrapper(instance);
+      ..bindingUtils = TestBindingWrapper(instance)
+      ..reportViewHierarchyIdentifiers = reportViewHierarchyIdentifiers;
     return SentryViewHierarchyEventProcessor(options);
   }
 }