Skip to content

Commit 7de383b

Browse files
authored
LoadConfigurationFromSection should RegisterConfigSettings without configuration assign (#627)
1 parent 0e69f8c commit 7de383b

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

src/NLog.Extensions.Logging/Config/SetupBuilderExtensions.cs

+1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ public static class SetupBuilderExtensions
1313
/// </summary>
1414
public static ISetupBuilder LoadConfigurationFromSection(this ISetupBuilder setupBuilder, Microsoft.Extensions.Configuration.IConfiguration configuration, string configSection = "NLog")
1515
{
16+
setupBuilder.SetupExtensions(ext => ext.RegisterExtensionsLogging(configuration));
1617
if (!string.IsNullOrEmpty(configSection))
1718
{
1819
var nlogConfig = configuration.GetSection(configSection);

src/NLog.Extensions.Logging/Config/SetupExtensionsBuilderExtensions.cs

+9
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,15 @@ public static class SetupExtensionsBuilderExtensions
1414
public static ISetupExtensionsBuilder RegisterConfigSettings(this ISetupExtensionsBuilder setupBuilder, IConfiguration configuration)
1515
{
1616
ConfigSettingLayoutRenderer.DefaultConfiguration = configuration;
17+
return setupBuilder.RegisterExtensionsLogging(configuration);
18+
}
19+
20+
internal static ISetupExtensionsBuilder RegisterExtensionsLogging(this ISetupExtensionsBuilder setupBuilder, IConfiguration configuration)
21+
{
22+
if (ConfigSettingLayoutRenderer.DefaultConfiguration is null)
23+
{
24+
ConfigSettingLayoutRenderer.DefaultConfiguration = configuration;
25+
}
1726
return setupBuilder.RegisterLayoutRenderer<ConfigSettingLayoutRenderer>("configsetting").RegisterLayoutRenderer<MicrosoftConsoleLayoutRenderer>("MicrosoftConsoleLayout");
1827
}
1928
}

0 commit comments

Comments
 (0)