Improved naming of discovered findings filter and add same for mitigated#10429
Conversation
…ded similar for mitigated field on api & UI
|
Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.
Note 🔴 Risk threshold exceeded. Adding a reviewer if one is configured in notification list: @mtesauro @grendel513 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 code changes in this pull request appear to be focused on enhancing the filtering and search functionality of the Defect Dojo application. The changes introduce a comprehensive set of filters that cover various models, including Findings, Engagements, Products, Endpoints, and more. The filters are designed to respect the user's permissions and only show the data that the user is authorized to access. The code also includes specialized filters for specific use cases, such as finding findings that are outside of their SLA or accepted findings. Additionally, the code includes API-specific filters and filters for the Engagement Survey feature. Overall, these changes provide a powerful and flexible filtering system that will greatly improve the user experience and data management capabilities of the Defect Dojo application. Files Changed:
Powered by DryRun Security |
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
DryRun Security SummaryThe pull request focuses on improving the filtering capabilities of the Defect Dojo application, particularly in the areas of security-related entities, with the introduction of new filters, enhanced tag-based filtering, performance optimizations, permissions-based filtering, and the addition of filters for metrics, reporting, and questionnaires/surveys. Expand for full summarySummary: The code changes in this pull request are focused on improving the filtering capabilities of the Defect Dojo application, particularly in the areas of security-related entities such as Findings, Engagements, Products, and Endpoints. The key changes include the introduction of new filters, enhanced tag-based filtering, performance optimizations, permissions-based filtering, and the addition of filters for metrics, reporting, and questionnaires/surveys. From an application security perspective, these changes are a positive step as they provide users with more granular control over the data they can access and analyze. The tag-based filtering, in particular, can be a powerful tool for categorizing and managing security findings. Additionally, the permissions-based filtering ensures that users can only access the data they are authorized to view, which is an important security consideration. Overall, these changes seem to be focused on improving the usability and functionality of the Defect Dojo application, with a strong emphasis on security-related features and capabilities. Files Changed:
Code AnalysisWe ran
Riskiness🔴 Risk threshold exceeded. We've notified @mtesauro, @grendel513. |
|
Conflicts have been resolved. A maintainer will review the pull request shortly. |
|
|
Hi @mtesauro, could you take a look here? It would be nice if we can merge this for the next release :-) |
Maffooch
left a comment
There was a problem hiding this comment.
@quirinziessler thank you for your patience here! I left a few comments for moving forward and getting these changes merged 😄
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Co-authored-by: Cody Maffucci <46459665+Maffooch@users.noreply.github.com>
Co-authored-by: Cody Maffucci <46459665+Maffooch@users.noreply.github.com>
Co-authored-by: Cody Maffucci <46459665+Maffooch@users.noreply.github.com>
|
Conflicts have been resolved. A maintainer will review the pull request shortly. |
|
@Maffooch added your suggestions to the MR. Sorry for the late reaction. Should be good to go from now. |



This PR updates the naming of filters related to #10401 to be a bit more precise.
Also it adds the same filter options for the "mitigated" field.