Add IHostApplicationBuilder support

[mikezw]

Hi, I add a support of IHostApplicationBuilder, so we can use IHostApplicationBuilder.UseNLog() after net8.0, but there is an issue when run bothUseNLog_LoadConfigurationFromSection and IHostApplicationBuilder_UseNLog_LoadConfigurationFromSection in tests. one will failed, Could anyone help take a look？

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.39%. Comparing base (165b24f) to head (727439b).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #780      +/-   ##
==========================================
- Coverage   81.58%   81.39%   -0.19%     
==========================================
  Files          19       19              
  Lines        1765     1774       +9     
  Branches      315      315              
==========================================
+ Hits         1440     1444       +4     
- Misses        182      187       +5     
  Partials      143      143              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[snakefoot]

I think it looks great. And it seems you have solved the issue with global-state bleeding into the next test (by clearing global state)

One could consider to introduce UseNLog(...)-methods that supports isolated LogFactory-instance, instead of the global LogManager-singleton.

public static IHostApplicationBuilder UseNLog(this IHostApplicationBuilder builder, NLogProviderOptions options, Func<IServiceProvider, LogFactory> factoryBuilder)

[snakefoot]

Do you expect any more changes, or is it ready to merge?

[mikezw]

Hi, I see the codes here
I can add the extensions but I have no idea on how to test it, or you can merge this and I will open another pr to add

public static IHostApplicationBuilder UseNLog(this IHostApplicationBuilder builder, NLogProviderOptions options, Func<IServiceProvider, LogFactory> factoryBuilder)
public static IHostBuilderr UseNLog(this IHostBuilder builder, NLogProviderOptions options, Func<IServiceProvider, LogFactory> factoryBuilder)

And I have a question, what will happen after I add the factoryBuilder in above api and ILoggingBuilder.AddNLog(factoryBuilder), are these two methods equivalent?

Maybe in IHostApplicationBuilder we can use the IHostApplicationBuilder.Logging property whitch type is ILoggingBuilder

[mikezw]

Hi, I was defeated by these providers, factories and builders. It might be a little difficult for me.

[snakefoot]

Yes the extension-logic is very convoluted, with the support for dependency resolving combined with applying dynamic configuration. I will merge this pull-request, and then see if I can figure out adding support for isolated LogFactory.

[snakefoot]

@mikezw I'm planning to get this released with NLog v5.5, before starting to update NLog.Extensions.Logging + NLog.Web.AspNetCore for NLog v6 RTM (making breaking changes)

I expect to release NLog v5.5 within the next 1-2 weeks. I hope this works for you?

P.S. Created #782 with support for isolated LogFactory

[mikezw]

Thank you very much. I will use it as soon as it is released

[snakefoot]

NLog.Extensions.Logging v5.5 has been released to nuget:

• https://www.nuget.org/packages/NLog.Extensions.Logging/5.5.0#releasenotes-body-tab

Thank you again for the contribution.