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

Upgrade to log4j2 #8

Closed
wants to merge 5 commits into from
Closed

Conversation

vaitkus
Copy link
Contributor

@vaitkus vaitkus commented Dec 20, 2021

This PR resolves issue #4.

@egonw egonw requested a review from petermr December 20, 2021 18:34
@egonw
Copy link
Contributor

egonw commented Dec 20, 2021

Looks good to me.

@egonw egonw self-requested a review December 20, 2021 18:34
@vaitkus
Copy link
Contributor Author

vaitkus commented May 19, 2024

@egonw, I guess this PR got superseded by commit 9b2db22?

@egonw
Copy link
Contributor

egonw commented May 19, 2024

I guess this PR got superseded by commit 9b2db22?

No, I don't think so. You actually upgraded the code; I just used the compatibility layer. I think I can still apply the rest of your PR.

@egonw egonw self-assigned this May 19, 2024
@egonw egonw added the enhancement New feature or request label May 19, 2024
@egonw egonw requested review from egonw and removed request for petermr May 19, 2024 19:44
@egonw
Copy link
Contributor

egonw commented May 23, 2024

Resolved the conflicts locally. Merged in.

@egonw egonw closed this May 23, 2024
@vaitkus vaitkus deleted the upgrade-to-log4j2 branch May 23, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants