-
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
🐛 fix Bump ruff from 0.7.2 to 0.7.3 #11224
🐛 fix Bump ruff from 0.7.2 to 0.7.3 #11224
Conversation
DryRun Security SummaryThe pull request focuses on updating and enhancing the serializers in the Defect Dojo API version 2, introducing a wide range of new serializers and updates to existing ones, covering various aspects of the application's functionality, and while the changes do not directly introduce any obvious security vulnerabilities, it is important to ensure that the new serializers and their associated data handling processes are thoroughly reviewed and tested to identify and address any potential security issues. Expand for full summarySummary: The changes in this pull request appear to be focused on updating and enhancing the serializers in the Defect Dojo API version 2. The changes introduce a wide range of new serializers and updates to existing ones, covering various aspects of the application's functionality, such as findings management, test management, JIRA and Sonarqube integration, user and group management, and more. From an application security perspective, these changes do not directly introduce any obvious security vulnerabilities. The updates to the serializers are primarily focused on improving the functionality and flexibility of the API, which is a positive step for the application's overall security posture. However, it is important to ensure that the new serializers and their associated data handling processes are thoroughly reviewed and tested to identify and address any potential security issues. Files Changed:
Code AnalysisWe ran
Riskiness🔴 Risk threshold exceeded. We've notified @mtesauro, @grendel513. |
Co-authored-by: Charles Neill <[email protected]>
I applied your suggestion @cneill |
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
#11222