-
Notifications
You must be signed in to change notification settings - Fork 1.6k
nessus: parse more fields #12247
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
nessus: parse more fields #12247
Conversation
This pull request involves potential information exposure in reference expansion, modifications to EPSS score processing, and the presence of multiple vulnerabilities in a Nessus scan template, which could impact system security and data handling. 💭 Unconfirmed Findings (3)
All finding details can be found in the DryRun Security Dashboard. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
d470a88
to
303a23e
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
e56437a
to
08320cd
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
2a6d0df
to
65cb49c
Compare
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
Users reported on Slack that the Nessus CSV report can contain fields that are currently not parsed by DefectDojo:
Reference: https://owasp.slack.com/archives/C2P5BA8MN/p1743609798217049
This PR parses some more fields in the csv parse:
To me it's not clear what some of these fields, but probably anyone using Nessus will understand. So it's good to have these values in the description, references or severity_justification field.
We decided for now not to parse the CWE field that is present in the XREF fields as it would influence the hash code calculation which we would not be able to get aligned with older findings that did not have the cwe field parsed and stored.
Ruff required me to update all test cases.