You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2021/11/23 12:47:12 UTC

[GitHub] [kafka] dongjinleekr commented on pull request #7898: KAFKA-9366: Change log4j dependency into log4j2

dongjinleekr commented on pull request #7898:
URL: https://github.com/apache/kafka/pull/7898#issuecomment-976480514


   Hi @naanagon,
   
   > Upgrading from log4j 1.2.17 to log4j 2 can be done because of log4j 2 features but not for this([CVE-2019-17571](https://github.com/advisories/GHSA-2qrg-x229-3v8q))
   
   Agree. After reconsidering the issue, I concluded that [CVE-2019-17571](https://github.com/advisories/GHSA-2qrg-x229-3v8q) is rather a minor issue; It is only problematic only when the user tries to use the `SocketServer` appender.
   
   I think the primary reason for migration is log4j2 features like syntax. (For example, using log4j 1.x syntax is confusing since it is already obsolete several years ago.)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org