Implemented retest feature#114
Implemented retest feature#114thejohnbrown wants to merge 2 commits intoblacklanternsecurity:github-devfrom
Conversation
|
could you see about resolving the merge conflicts that this branch has with the github-dev branch? |
|
Is there anything else required to get this merged? |
TMDeal
left a comment
There was a problem hiding this comment.
The HTML report does not seem to render anymore, and just gives a 500 error. Seems to be about mismatching if/endif statements.
Also, for the HTML templates for FindingList components, those have changed quite a bit from master to github-dev. Due to some pagedjs shenanigans, I had to drastically change how this template was structured, and use pagedjs hooks to further accomplish those goals. The relevant code for the pagedjs hooks can be found in paged-overrides.js. So for each section in the finding table, it needs to follow this pattern or it will not render properly.
<div class='finding-content'>
<div class='finding-content-header category'>
Category
</div>
<div class='finding-content-body' style='font-weight: bold'>
{{ finding.categoryFull }}
</div>
</div>
category can be replaced with any other name you need, like retest for example. The changes are pretty significant, so if you need any assistance with the changes made, please let me know
Once those issues are resolved, this should be good to go.
There was a problem hiding this comment.
This component should additionally be able to filter by finding group, like what we do in FindingsList.py. That way we dont always have to show the status of every finding group all the time.
There was a problem hiding this comment.
This css addition is no longer necessary, since this info should now just be put in the HTML template directly.

I have implemented a retest feature into Writehat which allows you to record and present the results from any vulnerability retesting that might occur for that engagement.
The following things have been added: