-
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
Add new parser - Threat Composer #10795
Conversation
DryRun Security SummaryThe provided code changes focus on improving the integration and handling of the Threat Composer parser in the DefectDojo application, including updates to documentation, configuration files, threat modeling data handling, unit tests, and the introduction of threat modeling data files. Expand for full summarySummary: The provided code changes cover a range of updates to the DefectDojo application, primarily focused on improving the integration and handling of the Threat Composer parser. The changes include:
From an application security perspective, the changes appear to be focused on improving the security posture of the DefectDojo application by enhancing the handling of threat modeling data and ensuring the robustness of the Threat Composer parser. The addition of unit tests and the comprehensive threat modeling data files are particularly noteworthy, as they demonstrate a proactive approach to identifying and addressing potential security risks. Files Changed:
Code AnalysisWe ran Riskiness🟢 Risk threshold not exceeded. |
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.
Couple of ideas for improvements. But I like it in general.
706e490
to
cfcb467
Compare
I have added the settings.dist.py checksum, how often are unit tests run? |
@mtesauro, can you enable tests for this PR, please? |
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
cfcb467
to
a4b2023
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
a4b2023
to
c052c8b
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
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.
LGTM
Description
New parser for Threat Composer - simple threat modeling tool
Each threat is equivalent to a finding. Mitigations associated with a threat are included in the finding mitigation field.
For more information, Threat Composer