-
Notifications
You must be signed in to change notification settings - Fork 325
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
bump org.apache.logging.log4j:log4j-core to 2.20.0 #3033
Conversation
💚 CLA has been signed |
👋 @candrews Thanks a lot for your contribution! It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it. Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it. |
/test |
@candrews please note that I will not be able to merge this PR without you signing the CLA. |
I signed the CLA - do I need to do something to get that reflected in this PR? |
No, we're good! |
run elasticsearch-ci/docs |
What does this PR do?
Update the
org.apache.logging.log4j:log4j-core
dependency in apm-agent-plugins/apm-ecs-logging-plugin/pom.xml to the current latest version,2.20.0
2.14.1
is vulnerable to CVE-2021-44228.I know this dependency is of
scope
provided
meaning it won't actually be included in the agent artifact jar, but the fact that there is a vulnerable dependency in the dependency graph at all (even if it isn't used and doesn't actually matter) isn't great, and definitely upsets some security folks.Checklist