jbertram commented on a change in pull request #662:
URL: https://github.com/apache/activemq/pull/662#discussion_r781684921
##########
File path: pom.xml
##########
@@ -82,7 +80,8 @@
<junit-version>4.13.2</junit-version>
<hamcrest-version>1.3</hamcrest-version>
<karaf-version>4.2.10</karaf-version>
- <log4j-version>1.2.17</log4j-version>
+ <slf4j-version>1.7.30</slf4j-version>
+ <log4j-version>2.14.1</log4j-version>
Review comment:
@tallpsmith then presumably they won't be using any version of ActiveMQ
"Classic" in any capacity anyway since it has always used Log4j 1.x. In any
case, it's already been noted on this thread and on others that 5.17.0 will use
the latest Log4j 2.x. _Hope_ isn't necessary.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
