Skip to content

Conversation

pablosnt
Copy link
Contributor

@pablosnt pablosnt commented Oct 20, 2025

Description

This fixes #13468

Add condition to run user_has_permission for risk acceptance entities based on the product associated with the engagement of the accepted findings.

Test results

Successfully made all the API requests that were affected by the bug.

@dryrunsecurity
Copy link

DryRun Security

🔴 Risk threshold exceeded.

This pull request includes a sensitive edit to dojo/authorization/authorization.py that may affect guarded code paths; sensitive file paths and allowed authors can be configured in .dryrunsecurity.yaml. Please review the change carefully to ensure no unauthorized or risky modification was introduced.

🔴 Configured Codepaths Edit in dojo/authorization/authorization.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

closed/reopened to get the test suite unstuck.

@valentijnscholten valentijnscholten linked an issue Oct 20, 2025 that may be closed by this pull request
3 tasks
@valentijnscholten valentijnscholten added this to the 2.51.3 milestone Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No authorization implemented for class Risk_Acceptance

2 participants