search

logpresso / CVE-2021-44228-Scanner

Vulnerability scanner and mitigation patch for Log4j2 CVE-2021-44228
Apache License 2.0
852 stars 173 forks source link

fix six sonarqube issues #275

Closed k3it closed 2 years ago

k3it commented 2 years ago

some fixes for #268

I was not sure how to fix the last two bugs in PartitionLoader.java but was able to get a clean run for the first six issues.

xeraph commented 2 years ago

@k3it Thank you for PR. I will merge it in a few days. :D

k3it commented 2 years ago

I was able to build a public sonar report in sonarcloud from my fork. Here is the link to results

https://sonarcloud.io/summary/overall?id=k3it_CVE-2021-44228-Scanner

k3it commented 2 years ago

added changes as suggested for this PR and reverted pom.xml and build.xml sonar lines.

xeraph commented 2 years ago

@k3it Merged. Thank you for PR!