MinimumEventLevel and MinimumBreadcrumbLevel options ignored when configured through AddSentry extension

Package

Sentry.Extensions.Logging

.NET Flavor

.NET

.NET Version

6.0.0

OS

Windows

SDK Version

3.22.0

Self-Hosted Sentry Version

No response

Steps to Reproduce

Console app:

using var loggingFactory = LoggerFactory.Create(builder =>
{
    builder.AddSentry(o => {
        o.Dsn = "<your dsn here>";
        o.MinimumBreadcrumbLevel = LogLevel.Trace;
        o.MinimumEventLevel = LogLevel.Information;
        o.IsGlobalModeEnabled = true;
    });
});


var logger = loggingFactory.CreateLogger<Program>();

// should add a breadcrumb
logger.LogTrace("test trace");

// should send an event to Sentry.io, with the "test trace" breadcrumb above
logger.LogInformation("test information");

// should send a 2nd event to Sentry.io
logger.LogWarning("test warning");

Expected Result

Two events sent to Sentry.io. The first should include a breadcrumb “test trace”.

Actual Result

A single event is sent to Sentry.io, with a breadcrumb “test information”.

[mattjohnsonpint]

Changing my mind about this - it certainly is a bug, not an enhancement. The intent was always that MinimumEventLevel and MinimumBreadcrumbLevel could be set independently of any other logging configuration and would work correctly, but presently it is constrained by the default global log level, or the log level set for a particular component or logging category.

Fixed in #1992, so that the MinimumEventLevel and MinimumBreadcrumbLevel can be set lower than the default.

[mattjohnsonpint]

Ok, researching more, this was helpful (both “Default log level” and “Filter function” right below it). https://learn.microsoft.com/en-us/dotnet/core/extensions/logging#default-log-level

Thus, instead of SetMinimumLevel, we can use:

builder.AddFilter<SentryLoggerProvider>(_ => true);

That will send all logs to the SentryLogger regardless of level, then our own MinimumBreadcrumbLevel and MinimumEventLevel will always work as expected. I’ll see if we can add this to AddSentry so it’s the default, but you can add it safely to your own code as well.