Skip to content
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

NLogRequestLoggingMiddleware - Added support for custom ShouldLogRequest-delegate #783

Merged
merged 2 commits into from
May 29, 2022

Conversation

snakefoot
Copy link
Contributor

No description provided.

@snakefoot snakefoot added this to the 5.0.1 milestone May 29, 2022
@codecov-commenter
Copy link

codecov-commenter commented May 29, 2022

Codecov Report

Merging #783 (eb5ed6a) into master (55a7fca) will increase coverage by 3%.
The diff coverage is 65%.

@@          Coverage Diff          @@
##           master   #783   +/-   ##
=====================================
+ Coverage      63%    66%   +3%     
=====================================
  Files          48     48           
  Lines         790    797    +7     
  Branches      196    197    +1     
=====================================
+ Hits          498    524   +26     
+ Misses        223    196   -27     
- Partials       69     77    +8     
Impacted Files Coverage Δ
src/NLog.Web/NLogRequestLoggingModule.cs 59% <65%> (+59%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 55a7fca...eb5ed6a. Read the comment docs.

@sonarcloud
Copy link

sonarcloud bot commented May 29, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

80.4% 80.4% Coverage
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ASP.NET Core ASP.NET Core - all versions enhancement size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants