-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
SO-6223: Remove log4j 1.x jars completely #1315
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
...to avoid including apache log4j in the system
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥓
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 9.x #1315 +/- ##
============================================
- Coverage 48.46% 48.46% -0.01%
+ Complexity 14035 14033 -2
============================================
Files 1949 1949
Lines 95377 95377
Branches 11024 11024
============================================
- Hits 46226 46225 -1
+ Misses 46110 46109 -1
- Partials 3041 3043 +2 ☔ View full report in Codecov by Sentry. |
No description provided.