From 7ad1e638e6f269fe278d97b550797f8a8db22a2f Mon Sep 17 00:00:00 2001 From: Vitor Rodrigues Date: Thu, 27 Oct 2022 11:42:01 +0200 Subject: [PATCH] Re-use any existing ReloadableLogger It's possible that consumers have already set up a bootstrap logger themselves before Umbraco had a chance too, if that's the case we just reload it. --- .../Extensions/ServiceCollectionExtensions.cs | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/Umbraco.Web.Common/Extensions/ServiceCollectionExtensions.cs b/src/Umbraco.Web.Common/Extensions/ServiceCollectionExtensions.cs index 9a9d606644..c9cd08516d 100644 --- a/src/Umbraco.Web.Common/Extensions/ServiceCollectionExtensions.cs +++ b/src/Umbraco.Web.Common/Extensions/ServiceCollectionExtensions.cs @@ -96,11 +96,18 @@ public static class ServiceCollectionExtensions // Bootstrap logger setup /////////////////////////////////////////////// - LoggerConfiguration serilogConfig = new LoggerConfiguration() + Func serilogConfig = cfg => cfg .MinimalConfiguration(hostEnvironment, loggingConfig, umbracoFileConfiguration) .ReadFrom.Configuration(configuration); - Log.Logger = serilogConfig.CreateBootstrapLogger(); + if (Log.Logger is ReloadableLogger reloadableLogger) + { + reloadableLogger.Reload(serilogConfig); + } + else + { + Log.Logger = serilogConfig(new LoggerConfiguration()).CreateBootstrapLogger(); + } /////////////////////////////////////////////// // Runtime logger setup