Skip to content

Commit 4c826df

Browse files
authored
Merge pull request #93 from serilog/fix-flipped-conditional
Fix flipped conditional introduced in earlier provider support PR
2 parents 27ba52d + 2e63ca9 commit 4c826df

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/Serilog.AspNetCore/SerilogWebHostBuilderExtensions.cs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@ public static IWebHostBuilder UseSerilog(
8888
{
8989
if (builder == null) throw new ArgumentNullException(nameof(builder));
9090
if (configureLogger == null) throw new ArgumentNullException(nameof(configureLogger));
91+
9192
builder.ConfigureServices((context, collection) =>
9293
{
9394
var loggerConfiguration = new LoggerConfiguration();
@@ -105,13 +106,13 @@ public static IWebHostBuilder UseSerilog(
105106
ILogger registeredLogger = null;
106107
if (preserveStaticLogger)
107108
{
108-
// Passing a `null` logger to `SerilogLoggerFactory` results in disposal via
109-
// `Log.CloseAndFlush()`, which additionally replaces the static logger with a no-op.
110-
Log.Logger = logger;
109+
registeredLogger = logger;
111110
}
112111
else
113112
{
114-
registeredLogger = logger;
113+
// Passing a `null` logger to `SerilogLoggerFactory` results in disposal via
114+
// `Log.CloseAndFlush()`, which additionally replaces the static logger with a no-op.
115+
Log.Logger = logger;
115116
}
116117

117118
collection.AddSingleton<ILoggerFactory>(services =>

0 commit comments

Comments
 (0)