Rolf Kristensen

Results 263 comments of Rolf Kristensen

Resolved by #3585 with NLog v4.6.8. Notice NLog v5.2 marks `GetLogger(string name, Type loggerType)` as obsolete and redirects to `GetLogger(string name) where T : Logger`

@RomanSoloweow You are welcome to provide a pull-request with your ideas.Make sure to read https://github.com/NLog/NLog/issues/2119#issuecomment-302490558 before you start implementing a solution. **Update** For some unknown reason the issue was closed...

NLog v5.2.6 has been released - https://www.nuget.org/packages/NLog/5.2.6 Where [NLog FileTarget](https://github.com/NLog/NLog/wiki/File-target) has the new setting `WriteHeaderWhenInitialFileNotEmpty`. Thanks to @RomanSoloweow

Resolved by this commit for xUnit 2.6.6 + xUnit 2.7.0 (and newer) - https://github.com/xunit/xunit/commit/9db7d3053ed3ceb8ca8e679fc08d211026637db0

Thank you for spotting this. Much appreciated.

> Should we merge this in, or close and leave unmerged? Have we received any community feedback? I have not heard anything from anyone, even though we been poking those...

https://www.nuget.org/packages/NLog.Web.AspNetCore v5.2.1 has been released with support for `AspNetBufferingWrapper` on ASP.NET Core via `NLogBufferingTargetWrapperMiddleware`. - https://github.com/NLog/NLog/wiki/AspNetBufferingWrapper-target Thanks to @bakgerman that made it happen.

Maybe just a manual mock like this: ```c# HttpContext httpContext = new HttpContext(new MyWorkerRequest()); public class MyWorkerRequest : SimpleWorkerRequest { public MyWorkerRequest() :base("/", "/", "/", "", new StringWriter(CultureInfo.InvariantCulture)) { }...

I think it is still interesting to have diagnostics from the NLog-targets, and maybe also basic-statistics about logged-errors-per-minute-by-host-application.

Are the HealthChecks-dependencies not included by default when project is using: ```xml ``` I'm ok with only adding HealthCheck-support for only some platforms like NET6, can see the following screenshot:...