Skip to content

Commit 0faeefb

Browse files
authored
Merge pull request #113 from Sensarg22/dev
I have changed the registration of the LoggerProvider with using a factory for creating an instance.
2 parents d4f3333 + ae4991f commit 0faeefb

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

src/Serilog.Extensions.Logging/SerilogLoggingBuilderExtensions.cs

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
#if LOGGING_BUILDER
1616

1717
using System;
18+
using Microsoft.Extensions.DependencyInjection;
1819
using Microsoft.Extensions.Logging;
1920
using Serilog.Extensions.Logging;
2021

@@ -38,7 +39,15 @@ public static ILoggingBuilder AddSerilog(this ILoggingBuilder builder, ILogger l
3839
{
3940
if (builder == null) throw new ArgumentNullException(nameof(builder));
4041

41-
builder.AddProvider(new SerilogLoggerProvider(logger, dispose));
42+
if (dispose)
43+
{
44+
builder.Services.AddSingleton<ILoggerProvider, SerilogLoggerProvider>(services => new SerilogLoggerProvider(logger, true));
45+
}
46+
else
47+
{
48+
builder.AddProvider(new SerilogLoggerProvider(logger));
49+
}
50+
4251
builder.AddFilter<SerilogLoggerProvider>(null, LogLevel.Trace);
4352

4453
return builder;

0 commit comments

Comments
 (0)