search

1modm / petereport

PeTeReport is an open-source application vulnerability reporting tool.
https://1modm.github.io/petereport/
BSD 3-Clause "New" or "Revised" License
465 stars 129 forks source link

Screenshots are not added to the appendix in reports #15

Closed eMVee-NL closed 2 years ago

eMVee-NL commented 2 years ago

I added a screenshot to a finding in the appendix. But the screenshot and the finding are not shown in reports (PDF and HTML as example). The finding and screenshot could be viewed within PeTeReport.

I've attached screenshots of my test environment, yes with fake data to test.

All Topics within the Appendix in PeTeReport appendix - overview screenshot

Screenshot is shown when the appendex is viewed within PeTeReport appendix - screenshot

In the HTML report the appendix is not present for the Screenshot SQL injection appendix - no screenshot in HTML report

In the PDFreport the appendix is not present for the Screenshot SQL injection appendix - no screenshot in PDF report

eMVee-NL commented 2 years ago

Update....

In the old version from 11/Nov/2021 the appendix does work...

see screenshots of my first test environment

PDF report with screenshot in appendix old version - appendix

HTM report with screenshot in appendix old version - appendix - HTM t

1modm commented 2 years ago

@mvdvaart I think I found the issue, the appendix linked to a finding with None severity was not showed correctly, that should be fixed in 0.5 release. Take a look and let me know. Now Findings (and their Appendix) with severity None are not added to the report.

eMVee-NL commented 2 years ago

@1modm , I will test it somewhere this weekend. Was a bit busy last week

eMVee-NL commented 2 years ago

I have tested this in version 0.5 and I have to say sadly it did not work yet.. The screenshots which are added to the appendix related to a finding are not shown in appendix. On the website the appendix is shown..

I've attached some screenshots....

Appendix 1 Appendix 2 Appendix 3 Appendix 4

1modm commented 2 years ago

Solved, found the issue in the appendix generation code. Thanks for reporting!