logpresso / CVE-2021-44228-Scanner

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

in CSV Report first column isn't formatted like the others #185

Closed mhochhaus closed 2 years ago

mhochhaus commented 2 years ago

We want to use the csv Report and import the generated Files automatically into a database. The first column with the hostname doesn't have apostrophes around the name.

CLIENT-246,"/Applications/Arduino.app/Contents/Java/log4j-core-2.12.0.jar","","Log4j 2","2.12.0","CVE-2021-44228","VULNERABLE","","2021-12-21 12:01:57"

Can you please format all columns in the same way.

xeraph commented 2 years ago

@mhochhaus Sure. I will fix it.

mhochhaus commented 2 years ago

Thx a lot mate.

xeraph commented 2 years ago

@mhochhaus Do you need double quotes in CSV header line? Current version is:

Hostname,Path,Entry,Product,Version,CVE,Status,Fixed,Detected at
xeraph commented 2 years ago

@mhochhaus Would you test v2.5.1 release?

mhochhaus commented 2 years ago

We skip the header during import in Database. Just checked Version 2.5.1 and it looks good.

"Hostname","Path","Entry","Product","Version","CVE","Status","Fixed","Detected at"
"Client-246","/Applications/Arduino.app/Contents/Java/log4j-core-2.12.0.jar","","Log4j 2","2.12.0","CVE-2021-44228","VULNERABLE","","2021-12-21 14:23:26"
xeraph commented 2 years ago

@mhochhaus Thank you for test report!