-
Notifications
You must be signed in to change notification settings - Fork 45
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
Refactor AuthLogPlugin #860
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #860 +/- ##
==========================================
+ Coverage 75.59% 76.76% +1.17%
==========================================
Files 311 316 +5
Lines 26561 27272 +711
==========================================
+ Hits 20078 20935 +857
+ Misses 6483 6337 -146
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
66638fe
to
d5273dc
Compare
Small elaboration: Went for an approach where each record will always contain:
I then try to parse additional fields within the message based on the service that created the log line (where |
284df3c
to
13e8670
Compare
e5f6a0e
to
2d7e28e
Compare
When this PR is ready to be merged in |
Fixes #286