TheGroundZero / openvasreporting

OpenVAS Reporting: Convert OpenVAS XML report files to reports
Other
131 stars 45 forks source link

Vuln Version ! #51

Closed LaCapitainerie closed 6 months ago

LaCapitainerie commented 6 months ago

In this update i added Vuln Version number, fix some bug, and tested the update for all possible exports In order to continue working on it i take the liberty of typing all classes because it was pretty annoying to look at the definition and forgot where i was..

Excel By Vuln

image

Excel By Host

image

Word By Vuln

image
LaCapitainerie commented 6 months ago

Yes i forgot about that, fixed it 👍

TheGroundZero commented 6 months ago

Looks good. Thank you for your contribution! I don't use OpenVAS myself anymore so can't really compare. So if problems arise because of this change, I may need to forward people to you instead ;-)

Can you up the version to 1.6.0 and update the Changelog please? https://github.com/TheGroundZero/openvasreporting/blob/master/setup.cfg / https://github.com/TheGroundZero/openvasreporting/blob/master/docs/conf.py https://github.com/TheGroundZero/openvasreporting/blob/master/CHANGELOG / https://github.com/TheGroundZero/openvasreporting/blob/master/docs/changelog.rst

LaCapitainerie commented 6 months ago

Looks good. Thank you for your contribution! I don't use OpenVAS myself anymore so can't really compare. So if problems arise because of this change, I may need to forward people to you instead ;-)

Sure ! I'm currently working and trying to resolve all the issues that are still open, could you please close or add a message on those where you were speaking with the author of the issue ? (To know if the problem is still on)