Skip to content

Conversation

@kiblik
Copy link
Contributor

@kiblik kiblik commented Apr 5, 2025

Last iteration for builtin-open (PTH123)

@dryrunsecurity
Copy link

dryrunsecurity bot commented Apr 5, 2025

DryRun Security Summary

Unit test files in the unittests/tools directory were modernized by updating file opening methods to use path object .open() method instead of traditional open() function, with no security vulnerabilities detected.

Expand for full summary

Summary: Multiple unit test files in the unittests/tools directory have been updated to use a more modern Python file opening method, replacing traditional open() function with .open() method on path objects.

Security Findings:
• No direct security vulnerabilities were introduced across the reviewed files
• All patches maintain consistent UTF-8 encoding
• Changes represent a syntax modernization with no functional security implications

No concrete security vulnerabilities were found in these patches.

View PR in the DryRun Dashboard.

@Maffooch Maffooch modified the milestones: 2.45.1, 2.46.0 Apr 7, 2025
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

@Maffooch Maffooch merged commit 9fcf423 into DefectDojo:dev Apr 9, 2025
77 checks passed
@kiblik kiblik deleted the ruff/PTH123_3 branch April 10, 2025 19:40
Maffooch pushed a commit that referenced this pull request Apr 21, 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