Skip to content

Conversation

manuel-sommer
Copy link
Contributor

uwsgi-1 | File "/app/dojo/finding_group/views.py", line 288, in paginate_queryset
uwsgi-1 | paginator = Paginator(queryset, page_size)
uwsgi-1 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
uwsgi-1 | File "/usr/local/lib/python3.11/site-packages/django/core/paginator.py", line 46, in init
uwsgi-1 | self._check_object_list_is_ordered()
uwsgi-1 | File "/usr/local/lib/python3.11/site-packages/django/core/paginator.py", line 142, in _check_object_list_is_ordered
uwsgi-1 | warnings.warn(
uwsgi-1 | django.core.paginator.UnorderedObjectListWarning: Pagination may yield inconsistent results with an unordered object_list: <class 'dojo.models.Finding_Group'> QuerySet.

Copy link

dryrunsecurity bot commented Sep 5, 2025

DryRun Security

🔴 Risk threshold exceeded.

This pull request modifies a sensitive file (dojo/finding_group/views.py) and was flagged twice by the scanner as "Configured Codepaths Edit" (sensitive edits detected); the finding lists a risk threshold of "failing" but is not blocking, and allowed authors/sensitive paths can be configured in .dryrunsecurity.yaml.

🔴 Configured Codepaths Edit in dojo/finding_group/views.py
Vulnerability Configured Codepaths Edit
Description Sensitive edits detected for this file. Sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml.
🔴 Configured Codepaths Edit in dojo/finding_group/views.py
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.

@valentijnscholten
Copy link
Member

Thanks. Does this mean we lack some pageviews in the integration tests?

@Maffooch Maffooch requested review from dogboat and Jino-T September 5, 2025 14:31
@manuel-sommer
Copy link
Contributor Author

Thanks. Does this mean we lack some pageviews in the integration tests?

I have no clue, just ran into this error. Might be

@valentijnscholten
Copy link
Member

Is it an error or warning as I am unable to reproduce it.

@valentijnscholten
Copy link
Member

added some basic UI tests. the ruff stuff should be fixed soon on the bugfix branch.

@manuel-sommer
Copy link
Contributor Author

manuel-sommer commented Sep 5, 2025

It was an error. You can reproduce it, if you setup a new DefectDojo (docker system prune and docker volume prune)

@manuel-sommer
Copy link
Contributor Author

manuel-sommer commented Sep 5, 2025

Just reproduced it again:
grafik

@valentijnscholten valentijnscholten merged commit 5ada3e3 into DefectDojo:bugfix Sep 5, 2025
84 checks passed
@manuel-sommer manuel-sommer deleted the fix_finding_group branch September 5, 2025 22:08
@valentijnscholten valentijnscholten added this to the 2.50.1 milestone Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants