Skip to content

Conversation

@testaccount90009
Copy link
Contributor

autoWidth false on engagement/product to fix formatting. I originally added the changes here #13868 - but ran into issues with my system and had to work it out.

autoWidth false on engagement/product to fix formatting
@testaccount90009
Copy link
Contributor Author

@valentijnscholten thanks for your guidance and patience while I submit a new PR.

@dryrunsecurity
Copy link

DryRun Security

🔴 Risk threshold exceeded.

This pull request modifies two template files (dojo/templates/dojo/engagements_all.html and dojo/templates/dojo/product.html) that match configured sensitive code paths; the scanner flagged these as sensitive edits and notes allowed authors/paths can be configured in .dryrunsecurity.yaml. No blocking action was taken but the findings are marked at the failing risk threshold.

🔴 Configured Codepaths Edit in dojo/templates/dojo/engagements_all.html
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/templates/dojo/product.html
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.

@github-actions github-actions bot added the ui label Dec 12, 2025
@valentijnscholten
Copy link
Member

@testaccount90009 thanks! Can you add some screenshots?

@valentijnscholten valentijnscholten added this to the 2.53.2 milestone Dec 12, 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

@mtesauro mtesauro requested a review from Jino-T December 17, 2025 03:27
@valentijnscholten valentijnscholten merged commit e188bb4 into DefectDojo:bugfix Dec 17, 2025
148 of 149 checks passed
@testaccount90009
Copy link
Contributor Author

@testaccount90009 thanks! Can you add some screenshots?

@valentijnscholten Sorry I was away from my personal PC for some time. I'll make sure to add screenshots of any HTML edits next time. My apologies!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants