NLog.Web
NLog.Web copied to clipboard
NLog integration for ASP.NET & ASP.NET Core 2-8
This is so, I can assign this to myself as a documentation task, after I finish my other pull requests.
From https://github.com/NLog/NLog/issues/2806 Port to middleware. Current docs: https://github.com/nlog/nlog/wiki/AspNetBufferingWrapper-target
Attempt to move NLog.Web target wrapper to NLog.Web.AspNetCore Application must setup middleware `NLogBufferingTargetWrapperMiddleware`, which enables the use of target-wrapper `AspNetBufferingTargetWrapper` (`type="AspNetBufferingWrapper"`)
The pre-validation of whether active HttpContext is available provides very little value, but gives a performance hit.
It is silly that we have this code: ```c# public class DefaultHttpContextAccessor : IHttpContextAccessor { public HttpContextBase HttpContext { get { var httpContext = System.Web.HttpContext.Current; if (httpContext == null) return...
See [Package readme on NuGet.org docs](https://docs.microsoft.com/en-us/nuget/nuget-org/package-readme-on-nuget-org) and [MSBuild docs](https://docs.microsoft.com/en-us/nuget/reference/msbuild-targets#packagereadmefile) - [ ] enable snupgk. But currently leads to [NU5039](https://docs.microsoft.com/en-us/nuget/reference/errors-and-warnings/nu5039) -> 2021-07-24 not fixed yet - [x] wait for dotnet RTM...
Keep users from thinking that they need to call both: ```c# builder.AddNLog() builder.AddNLogWeb() ``` Avoid naming confusion with extension-methods from NLog.Extensions.Logging. Making it easier to explain users that they should...
Something similar to these: https://github.com/Xabaril/AspNetCore.Diagnostics.HealthChecks/tree/master/src Maybe something like this: - Setup static capture of NLog.InternalLogger-events on Warning and Error-Level. - Keeps tracks of alert count per NLog Target within the...