NLogLoggerProvider - Always load from appsettings when possible (#529)

NLog · Aug 30, 2021 · 14935c2 · 14935c2
1 parent c8dcd7e
commit 14935c2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/src/NLog.Extensions.Hosting/Extensions/ConfigureExtensions.cs b/src/NLog.Extensions.Hosting/Extensions/ConfigureExtensions.cs
@@ -48,17 +48,14 @@ private static void AddNLogLoggerProvider(IServiceCollection services, IConfigur
 
         private static NLogLoggerProvider CreateNLogLoggerProvider(IServiceProvider serviceProvider, IConfiguration configuration, NLogProviderOptions options)
         {
-            configuration = SetupConfiguration(serviceProvider, configuration);
             NLogLoggerProvider provider = new NLogLoggerProvider(options);
+            configuration = SetupConfiguration(serviceProvider, configuration);
             if (configuration != null)
             {
-                if (options == null)
-                {
-                    provider.Configure(configuration.GetSection("Logging:NLog"));
-                }
-
+                provider.Configure(configuration.GetSection("Logging:NLog"));
                 provider.TryLoadConfigurationFromSection(configuration);
             }
+
             return provider;
         }
 

diff --git a/src/NLog.Extensions.Logging/Extensions/ConfigureExtensions.cs b/src/NLog.Extensions.Logging/Extensions/ConfigureExtensions.cs
@@ -252,11 +252,7 @@ private static NLogLoggerProvider CreateNLogLoggerProvider(IServiceProvider serv
             configuration = SetupConfiguration(serviceProvider, configuration);
             if (configuration != null)
             {
-                if (options == null)
-                {
-                    provider.Configure(configuration.GetSection("Logging:NLog"));
-                }
-
+                provider.Configure(configuration.GetSection("Logging:NLog"));
                 provider.TryLoadConfigurationFromSection(configuration);
             }