-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
feat(logger): ability to skip logs based on user-defined logic #3593
Conversation
Is there anyone to review? |
I want to filter specific route, which contains param |
@palvaneh Please help to rebase the master branch. I will take it after CI passing. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3593 +/- ##
=======================================
Coverage 99.21% 99.21%
=======================================
Files 42 42
Lines 3175 3177 +2
=======================================
+ Hits 3150 3152 +2
Misses 17 17
Partials 8 8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
It's a backward-compatible update that let users skip logs based on customized logic.
Gin's
LoggerConfig
only allows configuringSkipPaths
. It might not be enough when users need to skip logging based on different params like full path or status code.