-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Make endpoint names visible in reports #10230
Make endpoint names visible in reports #10230
Conversation
Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.
Note 🟢 Risk threshold not exceeded. Change Summary (click to expand)The following is a summary of changes in this pull request made by me, your security buddy 🤖. Note that this summary is auto-generated and not meant to be a definitive list of security issues but rather a helpful summary from a security perspective. Summary: The provided code change appears to be an update to the The key changes include modifications to the CSS class for the panel containing the endpoint information, the display of detailed finding information (such as severity, EPSS score/percentile, status, risk acceptance, discovery date, age, reporter, mitigation details, CWE, and Dojo ID), the use of Markdown rendering for various fields, the ability to display sample HTTP requests and responses, finding images, and finding notes. These enhancements help to make the security findings report more visually appealing, informative, and actionable for security teams. Files Changed:
Powered by DryRun Security |
…me panel on reports so it can be seen (previously white text on white background)
0e80bca
to
0ecb889
Compare
Signed-off-by: DefectDojo <[email protected]>
This reverts commit f89fb27.
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.
Approved
Description
This patch adds a class to the endpoint name entry on a report so it can be seen; previously the white text was invisible on the white background, this adds the panel color so it can be seen.
Test results
Before (highlighted for visibility):
After:
[sc-6083]