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

Remove Log4j API 3.x #2325

Closed
wants to merge 4 commits into from
Closed

Conversation

ppkarwasz
Copy link
Contributor

This PR closes #2290.

This is a long-term goal. Currently this draft replaces some parts of log4j-api and log4j-api-test to their 2.x versions. Ultimately the log4j-api and log4j-api-test modules will be removed altogether.

This task also requires for the log4j-to-slf4j module to be rolled back to its 2.x version. The only differences between the two version are the use of Recycler versus ThreadLocal.

In order for Log4j Core 3.x not to depend on the utility classes in
Log4j API 2.x, we provide a new `AbstractLogger` in `log4j-sdk`.
After some minor corrections flow message methods are also immediately
inlineable.
@ppkarwasz ppkarwasz force-pushed the fix/rollback-log4j-api branch from 42a76d2 to a7c3282 Compare February 27, 2024 14:06
@ppkarwasz ppkarwasz force-pushed the fix/rollback-log4j-api branch from a7c3282 to 72d1120 Compare February 27, 2024 15:03
@ppkarwasz ppkarwasz self-assigned this Feb 29, 2024
@ppkarwasz ppkarwasz added the STF-Milestones Milestones funded by the Sovereign Tech Fund label Feb 29, 2024
@ppkarwasz ppkarwasz added this to the 3.x milestone Feb 29, 2024
@ppkarwasz
Copy link
Contributor Author

Before merging this, we need to remove TestListLogger from log4j-kit and replace it with the 2.x version of StatusLogger.

@ppkarwasz ppkarwasz closed this Mar 25, 2024
@ppkarwasz ppkarwasz deleted the fix/rollback-log4j-api branch March 25, 2024 08:47
@ppkarwasz ppkarwasz restored the fix/rollback-log4j-api branch March 26, 2024 05:06
@vy vy removed the STF-Milestones Milestones funded by the Sovereign Tech Fund label May 31, 2024
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.

2 participants