You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/01/08 03:44:12 UTC
[GitHub] [pulsar-manager] nicknezis opened a new pull request #438: Updated to fix Log4J security vulnerabilities
nicknezis opened a new pull request #438:
URL: https://github.com/apache/pulsar-manager/pull/438
### Motivation
Details at https://logging.apache.org/log4j/2.x/security.html
Description of CVE-2021-45105:
> Apache Log4j2 versions 2.0-alpha1 through 2.16.0 did not protect from uncontrolled recursion from self-referential lookups. When the logging configuration uses a non-default Pattern Layout with a Context Lookup (for example, $${ctx:loginId}), attackers with control over Thread Context Map (MDC) input data can craft malicious input data that contains a recursive lookup, resulting in a StackOverflowError that will terminate the process. This is also known as a DOS (Denial of Service) attack.
*Explain here the context, and why you're making that change. What is the problem you're trying to solve.*
### Modifications
This PR applies the Log4j patch provided by the Spring Boot team. It forces the version of Log4J to 2.17.1.
--
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: dev-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-manager] eolivelli commented on pull request #438: Updated to fix Log4J security vulnerabilities
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #438:
URL: https://github.com/apache/pulsar-manager/pull/438#issuecomment-1007969451
Ci failed.
Can you please take a look?
--
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: dev-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-manager] nicknezis commented on pull request #438: Updated to fix Log4J security vulnerabilities
Posted by GitBox <gi...@apache.org>.
nicknezis commented on pull request #438:
URL: https://github.com/apache/pulsar-manager/pull/438#issuecomment-1008307363
Looks like CI passed this time. I was not able to build locally before or after the change I'm introducing. But I did verify that the Gradle dependencies were updated to point to the new log4j. I was getting similar error during `./gradlew build` as I see in the failed output of #420.
Having a release with this update would be greatly appreciated because at the moment we are not allowed to use Pulsar Manager.
--
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: dev-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-manager] tuteng merged pull request #438: Updated to fix Log4J security vulnerabilities
Posted by GitBox <gi...@apache.org>.
tuteng merged pull request #438:
URL: https://github.com/apache/pulsar-manager/pull/438
--
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: dev-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org