You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/04/11 04:10:48 UTC
[GitHub] [pulsar] RobertIndie opened a new pull request, #15109: [fix][security] Remove log4j for CVE-2022-23307
RobertIndie opened a new pull request, #15109:
URL: https://github.com/apache/pulsar/pull/15109
### Motivation
[CVE-2022-23307](https://www.cvedetails.com/cve/CVE-2022-23307/) is a critical severity in Log4j 1.x.
Since log4j is currently an unused dependency in the project, we'd better remove log4j
```
[INFO] --- maven-dependency-plugin:3.1.1:analyze (default-cli) @ buildtools ---
[WARNING] Used undeclared dependencies found:
[WARNING] org.slf4j:slf4j-api:jar:1.7.25:compile
[WARNING] Unused declared dependencies found:
[WARNING] org.yaml:snakeyaml:jar:1.30:compile
[WARNING] org.apache.ant:ant:jar:1.10.12:compile
[WARNING] com.google.guava:guava:jar:31.0.1-jre:compile
[WARNING] com.google.inject:guice:jar:4.2.3:compile
[WARNING] junit:junit:jar:4.13.1:compile
[WARNING] org.apache.logging.log4j:log4j-api:jar:2.17.1:compile
[WARNING] org.apache.logging.log4j:log4j-core:jar:2.17.1:compile
[WARNING] org.apache.logging.log4j:log4j-slf4j-impl:jar:2.17.1:compile
[WARNING] org.apache.logging.log4j:log4j-1.2-api:jar:2.17.1:compile
[WARNING] org.slf4j:jcl-over-slf4j:jar:1.7.32:compile
```
### Modifications
* Remove log4j 1.x
### Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
### Documentation
- [x] `no-need-doc`
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] RobertIndie commented on pull request #15109: [fix][security] Remove log4j for CVE-2022-23307
Posted by GitBox <gi...@apache.org>.
RobertIndie commented on PR #15109:
URL: https://github.com/apache/pulsar/pull/15109#issuecomment-1094844956
/pulsarbot run-failure-checks
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] codelipenghui merged pull request #15109: [fix][security] Remove log4j for CVE-2022-23307
Posted by GitBox <gi...@apache.org>.
codelipenghui merged PR #15109:
URL: https://github.com/apache/pulsar/pull/15109
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] dave2wave closed pull request #15109: [fix][security] Remove log4j for CVE-2022-23307
Posted by GitBox <gi...@apache.org>.
dave2wave closed pull request #15109: [fix][security] Remove log4j for CVE-2022-23307
URL: https://github.com/apache/pulsar/pull/15109
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] dave2wave commented on pull request #15109: [fix][security] Remove log4j for CVE-2022-23307
Posted by GitBox <gi...@apache.org>.
dave2wave commented on PR #15109:
URL: https://github.com/apache/pulsar/pull/15109#issuecomment-1094534346
> ### Motivation
> Currently, there are a lot of security issues in Log4j 1.x. Including [CVE-2021-4104](https://github.com/advisories/GHSA-fp5r-v3w9-4333), CVE-2020-9493, [CVE-2022-23307](https://github.com/advisories/GHSA-f7vh-qwp3-x37m), [CVE-2022-23305](https://github.com/advisories/GHSA-65fg-84f6-3jq3), [CVE-2019-17571](https://github.com/advisories/GHSA-2qrg-x229-3v8q), [CVE-2022-23302](https://github.com/advisories/GHSA-w9p3-5cr8-m3jj)
>
> [CVE-2022-23307](https://www.cvedetails.com/cve/CVE-2022-23307/) is a critical severity in Log4j 1.x.
>
> See https://github.com/apache/pulsar/runs/5965626775?check_suite_focus=true
>
> Since log4j is currently an unused dependency in the project, we'd better remove log4j
>
> ```
> [INFO] --- maven-dependency-plugin:3.1.1:analyze (default-cli) @ buildtools ---
> [WARNING] Used undeclared dependencies found:
> [WARNING] org.slf4j:slf4j-api:jar:1.7.25:compile
> [WARNING] Unused declared dependencies found:
> [WARNING] org.yaml:snakeyaml:jar:1.30:compile
> [WARNING] org.apache.ant:ant:jar:1.10.12:compile
> [WARNING] com.google.guava:guava:jar:31.0.1-jre:compile
> [WARNING] com.google.inject:guice:jar:4.2.3:compile
> [WARNING] junit:junit:jar:4.13.1:compile
> [WARNING] org.apache.logging.log4j:log4j-api:jar:2.17.1:compile
> [WARNING] org.apache.logging.log4j:log4j-core:jar:2.17.1:compile
> [WARNING] org.apache.logging.log4j:log4j-slf4j-impl:jar:2.17.1:compile
> [WARNING] org.apache.logging.log4j:log4j-1.2-api:jar:2.17.1:compile
> [WARNING] org.slf4j:jcl-over-slf4j:jar:1.7.32:compile
> ```
>
> ### Modifications
> * Remove log4j 1.x
>
> ### Verifying this change
> This change is a trivial rework / code cleanup without any test coverage.
>
> ### Documentation
> * [x] `no-need-doc`
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] RobertIndie closed pull request #15109: [fix][security] Remove log4j for CVE-2022-23307
Posted by GitBox <gi...@apache.org>.
RobertIndie closed pull request #15109: [fix][security] Remove log4j for CVE-2022-23307
URL: https://github.com/apache/pulsar/pull/15109
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org