Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

[BUG] CVE-2017-5645 - upgrade org.apache.logging.log4j:log4j-core to version 2.8.2 or higher #141

Open
3 tasks
danielgrigg-skyfii opened this issue Aug 16, 2022 · 1 comment
Labels

Comments

@danielgrigg-skyfii
Copy link

Describe the bug
See https://www.cve.org/CVERecord?id=CVE-2017-5645

To Reproduce
See https://www.cve.org/CVERecord?id=CVE-2017-5645

Expected behavior
Not be vulnerable to https://www.cve.org/CVERecord?id=CVE-2017-5645

Screenshots
N/A

Code snippet
N/A

Environment
com.github.salesforce-marketingcloud:fuelsdk@1.6.0

The bug has the severity

  • [x ] Critical: The defect affects critical functionality or critical data. It does not have a workaround.
  • Major: The defect affects major functionality or major data. It has a workaround but is not obvious and is difficult.
  • Minor: The defect affects minor functionality or non-critical data. It has an easy workaround.
  • Trivial: The defect does not affect functionality or data. It does not even need a workaround. It does not impact productivity or efficiency. It is merely an inconvenience.

Additional context

@zacthompson
Copy link

Inaccurate to say "It does not have a workaround":
https://logging.apache.org/log4j/2.x/security.html#fixed-in-log4j-2-8-2-java-7

Java 6 users should avoid using the TCP or UDP socket server classes, or they can manually backport the security fix commit from 2.8.2.

Since this codebase only uses the Logger interface, there's nothing in here that would be directly affected by that CVE - users would have to be explicitly adding references in their own applications in order to get hit by the bug.

However, updating to a more recent version of log4j would eliminate the risk to users, at the small cost of dropping support for Java 6, which seems like a reasonable tradeoff in 2023.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants