disable logs per default

dart/lib/src/sentry_client.dart

@@ -490,6 +490,10 @@ class SentryClient {
     SentryLog log, {
     Scope? scope,
   }) async {
+    if (!_options.enableLogs) {
+      return;
+    }
+
     log.attributes['sentry.sdk.name'] = SentryLogAttribute.string(
       _options.sdk.name,
     );
@@ -518,7 +522,7 @@ class SentryClient {
     } else if (propagationContext != null) {
       log.traceId = propagationContext.traceId;
     }
-
+    
     // TODO: Batch in separate PR, so we can send multiple logs at once.
     final envelope = SentryEnvelope.fromLogs([log], _options.sdk);
 

dart/lib/src/sentry_options.dart

@@ -15,6 +15,7 @@ import 'sentry_exception_factory.dart';
 import 'sentry_stack_trace_factory.dart';
 import 'transport/noop_transport.dart';
 import 'version.dart';
+import 'package:meta/meta.dart' as meta;
 
 // TODO: shutdownTimeout, flushTimeoutMillis
 // https://api.dart.dev/stable/2.10.2/dart-io/HttpClient/close.html doesn't have a timeout param, we'd need to implement manually
@@ -531,6 +532,9 @@ class SentryOptions {
   /// This is opt-in, as it can lead to existing exception beeing grouped as new ones.
   bool groupExceptions = false;
 
+  @meta.experimental
+  bool enableLogs = false;
+
   SentryOptions({String? dsn, Platform? platform, RuntimeChecker? checker}) {
     this.dsn = dsn;
     if (platform != null) {

dart/test/sentry_client_test.dart

@@ -1723,7 +1723,18 @@ void main() {
       );
     }
 
+    test('disabled by default', () async {
+      final client = fixture.getSut();
+      final log = givenLog();
+
+      await client.captureLog(log);
+
+      expect((fixture.transport).logs, isEmpty);
+    });
+
     test('should capture logs as envelope', () async {
+      fixture.options.enableLogs = true;
+
       final client = fixture.getSut();
       final log = givenLog();
       final logJson = log.toJson();
@@ -1742,6 +1753,7 @@ void main() {
     });
 
     test('should add additional info to attributes', () async {
+      fixture.options.enableLogs = true;
       fixture.options.environment = 'test-environment';
       fixture.options.release = 'test-release';
 
@@ -1800,6 +1812,8 @@ void main() {
     });
 
     test('should set trace id if there is a scope span', () async {
+      fixture.options.enableLogs = true;
+
       final client = fixture.getSut();
       final log = givenLog();
       final scope = Scope(fixture.options);
@@ -1817,6 +1831,8 @@ void main() {
     test(
         'should set trace id from propagation context if there is no scope span',
         () async {
+      fixture.options.enableLogs = true;
+
       final client = fixture.getSut();
       final log = givenLog();
       final scope = Scope(fixture.options);