Skip to content
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

String Filtering: Correct Typo For Reviewers #10353

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

Maffooch
Copy link
Contributor

@Maffooch Maffooch commented Jun 6, 2024

With the "Filter String Matching Optimization" mode enabled in system settings, filtering by reviewers is not possible as there is typo. The correct typo from reviewer to reviewers allows the filter class to access the reviewers field on the Finding model

[sc-6164]

@Maffooch Maffooch added the bugfix label Jun 6, 2024
Copy link

dryrunsecurity bot commented Jun 6, 2024

Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.

DryRun Security Status Findings
Configured Codepaths Analyzer 0 findings
Sensitive Files Analyzer 0 findings
AppSec Analyzer 0 findings
Authn/Authz Analyzer 1 finding
Secrets Analyzer 0 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 code changes in this pull request are focused on enhancing the filtering and search capabilities of the Defect Dojo application. The key changes include the addition of new filter fields for the reviewer and reviewer_contains fields in the FindingFilterWithoutObjectLookups class, as well as the addition of reviewers and reviewers_contains fields. These new fields allow users to search for findings based on the username of the reviewer or the usernames of the reviewers.

From an application security perspective, these changes are a positive addition as they provide more granular control over how users can search for and access findings. By allowing users to search for findings based on the reviewer and reviewer usernames, the application can better control access to sensitive information and ensure that only authorized users can view and interact with certain findings. Additionally, the use of the CharFilter class for these new fields ensures that the search is case-insensitive, making it easier for users to find the information they need.

Files Changed:

  • dojo/filters.py: This file defines several Django filter classes that are used to filter and search for various objects in the Defect Dojo application, such as findings, products, engagements, and more. The key changes in this file include the addition of new filter fields for the reviewer, reviewer_contains, reviewers, and reviewers_contains fields in the FindingFilterWithoutObjectLookups class. These new fields allow users to search for findings based on the username of the reviewer or the usernames of the reviewers.

Powered by DryRun Security

Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@mtesauro mtesauro merged commit bb1239a into DefectDojo:bugfix Jun 6, 2024
122 checks passed
@Maffooch Maffooch deleted the reviewer branch June 6, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants