Fixes #13555 AND logic not present when filtering Findings via two or more tags#13785
Fixes #13555 AND logic not present when filtering Findings via two or more tags#13785PoojasPatel013 wants to merge 10 commits intoDefectDojo:bugfixfrom PoojasPatel013:master
Conversation
🔴 Risk threshold exceeded.This pull request modifies dojo/filters.py, which the scanner flagged as a sensitive edit; sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. The change is marked at a failing risk threshold but is not set to block the merge.
🔴 Configured Codepaths Edit in
|
| Vulnerability | Configured Codepaths Edit |
|---|---|
| Description | Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. |
We've notified @mtesauro.
All finding details can be found in the DryRun Security Dashboard.
|
Thanks, please base the PR against |
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
|
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. |
|
Conflicts have been resolved. A maintainer will review the pull request shortly. |
Closes #13555
I have applied logic to filters.py, where OR logic is present. Do a review and address any required revisions. Glad if I am even a little helpful in building this amazing app!!
I have kept the existing code as is. And added AND logic with label Tags (AND)
Test results
Have performed manual verification using the UI with a local development instance:
Setup: Created findings with various tag combinations (e.g.,
pci,pentest, andcloud).Action: Tested the new
Tags (AND)filter withpciandpentest..Result: The new filter works as expected without disrupting the existing functionality.
Documentation
N/A (UI enhancement).