Skip to content
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

Vulnerable log4j 1.2.x #632

Closed
hkz-aarvesen opened this issue Dec 13, 2021 · 4 comments
Closed

Vulnerable log4j 1.2.x #632

hkz-aarvesen opened this issue Dec 13, 2021 · 4 comments

Comments

@hkz-aarvesen
Copy link

hkz-aarvesen commented Dec 13, 2021

Describe the bug

Log4j has a bad security vulneraility: CVE-2019-17571. Per the Apache site, the remediation is to upgrade to log4j 2 with the fix.

Both the 1.9.2 branch (which I use) and the main branch use the vulnerable library. Please upgrade to the latest version.

To Reproduce

  1. Build code
  2. Note that we see the old jar used
  3. The pom.xml files specify an outdated version. For example, server/pom.xml:
        <dependency>
            <groupId>log4j</groupId>
            <artifactId>log4j</artifactId>
            <version>1.2.17</version>
        </dependency>

Expected behavior

Please upgrade to log4j 2.15 or later. Sadly, there are incompatibilities between versions.

Desktop (please complete the following information):
N/A

Smartphone (please complete the following information):
N/A

Additional context
-According to this page, one can also set a JVM property to avoid this.- Further digging looks like that is for log4j 2.10+, so not helpful here.

@hkz-aarvesen
Copy link
Author

Okay - on further review, version 1.x is not vulnerable to this particular bug. It is however vulnerable to other bugs :)

Closig this issue now so that remains clear.

@jrivard
Copy link
Contributor

jrivard commented Dec 14, 2021

see #628.

@sahil-sardana
Copy link

Hello @jrivard Can you please confirm if log4j-1.2.17.jar is vulnerable to CVE-2022-23305?

Can we migrate to log4j 2 following the below document:
https://logging.apache.org/log4j/2.x/manual/migration.html

@jrivard
Copy link
Contributor

jrivard commented Mar 31, 2022

See #628

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

No branches or pull requests

3 participants