-
-
Notifications
You must be signed in to change notification settings - Fork 746
reports: Support nodeNames and systemic counts #6846
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
c557803 to
ca20f98
Compare
|
Great job! No new security vulnerabilities introduced in this pull requestUse @Checkmarx to reach out to us for assistance. Just send a PR comment with Examples: |
1aed29a to
6006739
Compare
|
Ready for review. It doesnt cover the High Level or Risk Confidence reports, but they look a bit harder (and might be broken) so I plan to address them later.. |
addOns/reports/src/main/zapHomeFiles/reports/traditional-html/report.html
Show resolved
Hide resolved
addOns/reports/src/test/java/org/zaproxy/addon/reports/ExtensionReportsUnitTest.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall
6006739 to
7ff4bda
Compare
7c9cccc to
d0ed53a
Compare
Signed-off-by: Simon Bennetts <[email protected]>
d0ed53a to
22b9aa3
Compare
|
Thank you! |

Still a couple of reports to finish..