If UseNLog and AddNLog are used, two logs will be written

[czd890]

NLog version: 4.6.6
NLog.Extensions.Logging: 1.5.2
NLog.Web.AspNetCore: 4.8.4
Platform: .NET Core 2

If builder.AddNLog(this.Configuration); and .UseNLog(); are used,
two same logs will be written.

WebHost.CreateDefaultBuilder()
....
..UseStartup<Startup>()
.UseNLog();

class Startup {
    public void ConfigureServices(IServiceCollection services)
    {
        NLog.LogManager.Configuration = new NLogLoggingConfiguration(this.Configuration.GetSection("NLog"));
        services.AddLogging(builder =>
        {
            builder.AddConfiguration(this.Configuration.GetSection("Logging"));
            builder.AddNLog(this.Configuration);
            builder.AddConsole();
        });

    }
}

{
"Logging": {
        "IncludeScopes": true,
        "LogLevel": {
            "Default": "Warning"
        },
        "NLog": {
            "IncludeScopes": true,
            "CaptureMessageTemplates": true,
            "CaptureMessageProperties": true
        }
    },
    "NLog": {
            "NLog.Web.AspNetCore": {
                "Assembly": "NLog.Web.AspNetCore"
            }
        },
        "internalLogLevel": "Error",
        "internalLogFile": "logs/nlog-internal.log",
        "throwConfigExceptions": "True",
        "targets": {
            "file": {
                "type": "File",
                "fileName": "logs/logs.log",
                "keepFileOpen": true,
                "concurrentWrites": true,
                "archiveAboveSize": 10485760,
                "archiveEvery": "Day",
                "archiveNumbering": "DateAndSequence",
                "archiveDateFormat": "yyyyMMdd",
                "archiveFileName": "logs/logs.{#}.log",
                "layout": "${longdate:universalTime=true} ${logger} ${uppercase:${level}} ${threadid} ${threadname} ${message} ${exception:format=ToString:exceptionDataSeparator=\r\n}\r\n   on ${aspnet-TraceIdentifier} ${aspnet-request-url:IncludeHost=true:IncludePort=true:IncludeQueryString=true}"
            },
            "blackhole": { "type": "Null" }
        },
        "rules": {
            "01-allLogs": {
                "logger": "*",
                "minLevel": "Trace",
                "writeTo": "file",
                "final": true
            }
        }
    }
}

[304NotModified]

Yes, I don't think we could change that easily.

[snakefoot]

Yes duplicate of #254

Can be solved by one of these ways:

• Create new option NLogProviderOptions.ReplaceLoggerProviders that causes UseNLog / AddNLog to clear previous LoggerProviders (before adding itself)
• Create new option NLogProviderOptions.ReplaceLoggerFactory that causes UseNLog / AddNLog to replace the default LoggerFactory with NLogLoggerFactory

PullRequests are always welcome

[repo-ranger]

⚠️ This has been marked to be closed in 7 days.

[snakefoot]

Discovered a different way for resolving this issue. Created PR #325