[GitHub] [kafka] cadonna commented on a diff in pull request #12117: MINOR: Note that slf4j-log4j in version 1.7.35+ should be used

[GitBox <gi...@apache.org>]

cadonna commented on code in PR #12117:
URL: https://github.com/apache/kafka/pull/12117#discussion_r887704920


##########
docs/upgrade.html:
##########
@@ -73,7 +73,13 @@ <h5><a id="upgrade_311_notable" href="#upgrade_311_notable">Notable changes in 3
         via Connect worker and/or connector configuration. Connect may enable idempotent producers
         by default in a future major release.</li>
     <li>Kafka has replaced log4j and slf4j-log4j12 with reload4j and slf4j-reload4j due to security concerns.
-        More information can be found at <a href"https://reload4j.qos.ch">reload4j</a>.</li>
+        This only affects modules that specify a logging backend (<code>connect-runtime</code> and <code>kafka-tools</code> are two such examples).

Review Comment:
   Good point! Since this is actually a cherry-pick, I would propose to get this merged to be consistent with the source of this cherry-pick and then we can look into enumerating the modules. WDYT?



-- 
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