From 54cdfa8b02ba4fecd6c92213fcaf4c776c330809 Mon Sep 17 00:00:00 2001 From: Andrew McLachlan Date: Wed, 31 Jan 2024 08:33:02 +1000 Subject: [PATCH] Fixed an issue with Serilog not respecting the default log level. --- src/Asm.Serilog/LoggingConfigurator.cs | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/Asm.Serilog/LoggingConfigurator.cs b/src/Asm.Serilog/LoggingConfigurator.cs index 89974bb..49bc273 100644 --- a/src/Asm.Serilog/LoggingConfigurator.cs +++ b/src/Asm.Serilog/LoggingConfigurator.cs @@ -40,16 +40,14 @@ public static LoggerConfiguration ConfigureLogging(LoggerConfiguration configura public static LoggerConfiguration ConfigureLogging(LoggerConfiguration loggerConfiguration, IConfiguration configuration, IHostEnvironment hostEnvironment) { - if (loggerConfiguration == null) throw new ArgumentNullException(nameof(loggerConfiguration)); - if (configuration == null) throw new ArgumentNullException(nameof(configuration)); - if (hostEnvironment == null) throw new ArgumentNullException(nameof(hostEnvironment)); + ArgumentNullException.ThrowIfNull(loggerConfiguration); + ArgumentNullException.ThrowIfNull(configuration); + ArgumentNullException.ThrowIfNull(hostEnvironment); loggerConfiguration .Enrich.FromLogContext() .Enrich.WithProperty("App", hostEnvironment.ApplicationName) .Enrich.WithProperty("Env", hostEnvironment.EnvironmentName) - .MinimumLevel.Information() - .MinimumLevel.Is(LogEventLevel.Information) .WriteTo.Trace() .WriteTo.Console() .WriteTo.ApplicationInsights(TelemetryConfiguration.CreateDefault(), TelemetryConverter.Traces); @@ -61,6 +59,12 @@ public static LoggerConfiguration ConfigureLogging(LoggerConfiguration loggerCon { foreach(IConfigurationSection child in logLevelConfig.GetChildren()) { + if (child.Key == "Default") + { + loggerConfiguration.MinimumLevel.Is(logLevelConfig.GetValue(child.Key)); + continue; + } + loggerConfiguration.MinimumLevel.Override(child.Key, logLevelConfig.GetValue(child.Key)); } }