Skip to content

refs #1 update log4j dep#11

Open
lutin11 wants to merge 4 commits into
timewalker74:masterfrom
lutin11:feature/fixup-log4j-cve
Open

refs #1 update log4j dep#11
lutin11 wants to merge 4 commits into
timewalker74:masterfrom
lutin11:feature/fixup-log4j-cve

Conversation

@lutin11
Copy link
Copy Markdown

@lutin11 lutin11 commented Oct 8, 2025

This MR fix the CVE-2019-17571

@lutin11 lutin11 force-pushed the feature/fixup-log4j-cve branch from e0eef81 to 6e40740 Compare October 8, 2025 08:41
@timewalker74
Copy link
Copy Markdown
Owner

Hello,
I'd rather not change dependencies compatibility on the current branch because some users are embedding the server runtime directly in their application code.

Do you really need some log4j2 specific feature or is your goal to get rid of detected CVEs ?
For now, I chose a simpler path which is replacing the log4j dependency by reload4j (which is a binary compatible, maintained fork of log4j 1.x)

Note that the client part of FFMQ is only using the commons-logging facade, which you can adapt to whatever logging backend you need by adding one more wrapper (jcl-over-slf4j)

In the future I may create a new branch to modernize a bit the code :
raise minimum JRE to 8, replace commons-logging by slf4j-api, ...

@lutin11
Copy link
Copy Markdown
Author

lutin11 commented Oct 13, 2025

Hello, I'd rather not change dependencies compatibility on the current branch because some users are embedding the server runtime directly in their application code.

Do you really need some log4j2 specific feature or is your goal to get rid of detected CVEs ? For now, I chose a simpler path which is replacing the log4j dependency by reload4j (which is a binary compatible, maintained fork of log4j 1.x)

Actualy, I don't need, the goal was just to get rid of teh CVE

Note that the client part of FFMQ is only using the commons-logging facade, which you can adapt to whatever logging backend you need by adding one more wrapper (jcl-over-slf4j)

In the future I may create a new branch to modernize a bit the code : raise minimum JRE to 8, replace commons-logging by slf4j-api, ...

Ok, super !
Thanks for your work

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