-
Notifications
You must be signed in to change notification settings - Fork 166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added layoutrenderer aspnet-request-has-posted-body and aspnet-request-stream-id #845
Conversation
Codecov Report
@@ Coverage Diff @@
## master #845 +/- ##
=====================================
+ Coverage 69% 71% +1%
=====================================
Files 63 64 +1
Lines 1157 1187 +30
Branches 286 299 +13
=====================================
+ Hits 804 837 +33
+ Misses 229 214 -15
- Partials 124 136 +12
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
src/NLog.Web.AspNetCore/LayoutRenderers/AspNetRequestBodySynchronousIOAllowedLayoutRenderer.cs
Outdated
Show resolved
Hide resolved
src/NLog.Web.AspNetCore/LayoutRenderers/AspNetRequestCanHaveBodyLayoutRenderer.cs
Outdated
Show resolved
Hide resolved
src/NLog.Web.AspNetCore/LayoutRenderers/AspNetRequestMaxBodySizeLayoutRenderer.cs
Outdated
Show resolved
Hide resolved
src/NLog.Web.AspNetCore/LayoutRenderers/AspNetRequestStreamIdLayoutRenderer.cs
Show resolved
Hide resolved
src/NLog.Web.AspNetCore/LayoutRenderers/AspNetRequestTransferFormatLayoutRenderer.cs
Outdated
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Created the following wiki-pages: https://github.com/NLog/NLog/wiki/AspNet-Request-StreamId-Layout-Renderer Thank you once again for a fine contribution. |
There should not be any more unless .NET completely changes HttpContext in .NET 8
There were a few interfaces in Microsoft.AspNetCore.Connections.Features and Microsoft.AspNetCore.Http.Features to be added.