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

Fix log4j2 CVE and update dependencies #34

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

xSAVIKx
Copy link

@xSAVIKx xSAVIKx commented Dec 13, 2021

In this PR I have fixed the recent Log4j2 CVE-2021-44228.

As part of the PR, I have migrated the app to the latest Gradle v7.3.1 and Java 17 as well as updated the dependencies.

Copy link
Collaborator

@armiol armiol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xSAVIKx LGTM.

Next time, let's discuss the changes first. The thing is that mc-java is currently under a heavy re-design. So things might not work exactly as one would expect.

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

Successfully merging this pull request may close these issues.

3 participants