feat: Logs are always enabled and SentryOptions.EnableLogs is ignored (#5504)

* feat: Logs are always enabled and `SentryOptions.EnableLogs` is ignored

Adding one of the logging integrations (`Sentry.Extensions.Logging`,
`Sentry.Serilog`, `Sentry.NLog`, `Sentry.Log4Net`) is already an explicit
opt-in to sending logs, so requiring `EnableLogs = true` on top of that was
just an extra hoop. Structured logs are now captured unconditionally.

`SentryOptions.EnableLogs` and `SentryTarget.EnableLogs` are marked obsolete
rather than removed: the getters always return `true`, the setters are
ignored, and existing configuration (including `"EnableLogs"` in
appsettings.json and `enableLogs` on the Serilog sink) still binds without
error. To drop logs, use `SetBeforeSendLog` and return `null`.

Resolves #5479

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* Remove redundant comments

Co-authored-by: James Crosswell <jamescrosswell@users.noreply.github.com>

* ref: Create a disabled structured logger for a disabled Hub

Belt-and-braces: `DefaultSentryStructuredLogger` asserts `hub.IsEnabled`, so
`Create` now honours that by construction and returns the disabled logger
otherwise. Restores the disabled-path coverage that the `EnableLogs` test
removed, keyed off the Hub instead of the option.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* test: Update Versioning snapshots for net8.0/net9.0

`WebIntegrationTests.Versioning` has per-TFM snapshots; only the net10.0 one
was regenerated locally (no net8/net9 runtimes on the dev machine). Taken
byte-for-byte from the CI artifacts, which produced an identical delta on
every platform.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* test: Assert only on transactions in PreFlightIgnoresTransaction

Review feedback: the hub isn't disabled, so logs are captured as usual —
excluding logs was the wrong way round. Assert directly that no transaction
envelope item is sent, which is the only thing the test needs to show.

Uses the existing EnvelopeItem.TryGetType() and TypeValueTransaction rather
than a hand-rolled header lookup.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

* Revert "ref: Create a disabled structured logger for a disabled Hub"

This reverts commit 12ec118.

---------

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

Author: jamescrosswell

benchmarks/Sentry.Extensions.Logging.Benchmarks/SentryStructuredLoggerBenchmarks.cs

@@ -19,7 +19,6 @@ public void Setup()
         SentryLoggingOptions options = new()
         {
             Dsn = DsnSamples.ValidDsn,
-            EnableLogs = true,
         };
         options.SetBeforeSendLog((SentryLog log) =>
         {

samples/Sentry.Samples.AspNetCore.Basic/Program.cs

@@ -19,10 +19,6 @@
     // Configure the minimum Log Level of Breadcrumbs and Events
     options.MinimumBreadcrumbLevel = LogLevel.Information;
     options.MinimumEventLevel = LogLevel.Error;
-
-    // This option enables Logs sent to Sentry
-    // Configure the minimum Log Level of Structured-Logs via e.g. "appsettings.json" and "appsettings.{HostEnvironment}.json"
-    options.EnableLogs = true;
 });
 
 var app = builder.Build();

samples/Sentry.Samples.AspNetCore.Grpc/appsettings.json

@@ -23,9 +23,7 @@
     "DiagnosticLevel": "Error",
     "DefaultTags": {
       "default-key-in-config": "default-value"
-    },
-    // Record log messages as Structured Logs (configure via "Logging" section)
-    "EnableLogs": true
+    }
   },
   "Logging": {
     "IncludeScopes": false,

samples/Sentry.Samples.AspNetCore.Mvc/appsettings.json

@@ -23,9 +23,7 @@
     "DiagnosticLevel": "Error",
     "DefaultTags": {
       "default-key-in-config": "default-value"
-    },
-    // Record log messages as Structured Logs (configure via "Logging" section)
-    "EnableLogs": true
+    }
   },
   "Logging": {
     "LogLevel": {

samples/Sentry.Samples.GenericHost/appsettings.json

@@ -13,7 +13,6 @@
     //"Dsn": "TODO: Configure your DSN here and uncomment this line",
     "MinimumBreadcrumbLevel": "Debug",
     "MinimumEventLevel": "Warning",
-    "SendDefaultPii": true, // Send user name and machine name
-    "EnableLogs": true // Send structured logs
+    "SendDefaultPii": true // Send user name and machine name
   }
 }

samples/Sentry.Samples.Google.Cloud.Functions/appsettings.json

@@ -13,7 +13,6 @@
     //"Dsn": "TODO: Configure your DSN here and uncomment this line",
     "MaxRequestBodySize": "Always",
     "SendDefaultPii": true,
-    "EnableTracing": true,
-    "EnableLogs" : true
+    "EnableTracing": true
   }
 }

samples/Sentry.Samples.ME.AI.AspNetCore/Program.cs

@@ -14,7 +14,6 @@
     options.DiagnosticLevel = SentryLevel.Debug;
     options.SampleRate = 1;
     options.TracesSampleRate = 1.0;
-    options.EnableLogs = true;
 });
 
 // This sample uses Microsoft.Extensions.AI.OpenAI

samples/Sentry.Samples.ME.Logging/Program.cs

@@ -24,8 +24,6 @@
         options.MinimumBreadcrumbLevel = LogLevel.Information; // It requires at least this level to store breadcrumb
         options.MinimumEventLevel = LogLevel.Error; // This level or above will result in event sent to Sentry
 
-        // This option enables Logs sent to Sentry.
-        options.EnableLogs = true;
         options.SetBeforeSendLog(static log =>
         {
             log.SetAttribute("attribute-key", "attribute-value");

samples/Sentry.Samples.Maui/MauiProgram.cs

@@ -35,7 +35,6 @@ public static MauiApp CreateMauiApp()
                 options.AttachScreenshot = true;
 
                 options.Debug = true;
-                options.EnableLogs = true;
                 options.SampleRate = 1.0F;
 
                 // The Sentry MVVM Community Toolkit integration automatically creates traces for async relay commands,
@@ -79,7 +78,7 @@ public static MauiApp CreateMauiApp()
                 fonts.AddFont("OpenSans-Semibold.ttf", "OpenSansSemibold");
             });
 
-        // Configure Logging, including Structured Logs sent to Sentry (see 'EnableLogs = true')
+        // Configure Logging, including Structured Logs sent to Sentry
         builder.Logging.AddFilter(null, LogLevel.Warning);
         builder.Logging.AddFilter("Sentry.Samples.Maui", LogLevel.Information);
 

samples/Sentry.Samples.NLog/NLog.config

@@ -22,8 +22,7 @@
             ignoreEventsWithNoException="False"
             includeEventDataOnBreadcrumbs="False"
             includeEventPropertiesAsTags="True"
-            minimumEventLevel="Error"
-            enableLogs="True">
+            minimumEventLevel="Error">
 
       <!-- Advanced options can be configured here-->
       <options