-
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
Improved naming of discovered findings filter and add same for mitigated #10429
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. |
Quality Gate passedIssues Measures |
Hi @mtesauro, could you take a look here? It would be nice if we can merge this for the next release :-) |
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.
@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 <[email protected]>
Co-authored-by: Cody Maffucci <[email protected]>
Co-authored-by: Cody Maffucci <[email protected]>
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. |
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
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.