Ruff: Fix Ruff FURB189 on bugfix

[manuel-sommer]

No description provided.

[dryrunsecurity]

DryRun Security Summary

The pull request contains routine updates to the serializer classes in the Defect Dojo application's API, including changes to the RequestResponseDict and RequestResponseSerializerField classes to improve the handling and validation of request/response data, which can help prevent potential security issues related to improper input validation.

Expand for full summary

Summary:

The code changes in this pull request appear to be a routine update to the serializer classes in the Defect Dojo application's API. The key changes include updating the RequestResponseDict class to inherit from collections.UserList instead of list, and improving the error handling and robustness of the RequestResponseSerializerField class when handling invalid JSON input.

From an application security perspective, these changes do not introduce any significant security concerns. Serializers play an important role in handling and validating the data sent to and from the API, which is a critical security consideration. The updates made in this pull request seem to improve the overall handling and validation of request/response data, which can help prevent potential security issues related to improper input validation.

Files Changed:

• dojo/api_v2/serializers.py: This file contains the serializer classes for the Defect Dojo application's API. The key changes include:
  1. The RequestResponseDict class has been updated to inherit from collections.UserList instead of list, likely improving the handling of request/response data.
  2. The RequestResponseSerializerField class has been updated to handle invalid JSON input more robustly, with better error messages.

Overall, these changes appear to be routine updates to the serializer classes, focusing on improving the handling of request/response data and input validation. There are no obvious security-related concerns in the changes provided.

Code Analysis

We ran 9 analyzers against 1 file and 1 analyzer had findings. 8 analyzers had no findings.

Analyzer	Findings
Configured Codepaths Analyzer	7 findings

Riskiness

🔴 Risk threshold exceeded.

We've notified @mtesauro, @grendel513.

View PR in the DryRun Dashboard.

Creating merge conflict