Skip to content

Fix #8056: Validate solution presence in all Vulnerabilities#7472

Closed
gmrnlg1971 wants to merge 1 commit into
zaproxy:mainfrom
gmrnlg1971:fix-issue-8056
Closed

Fix #8056: Validate solution presence in all Vulnerabilities#7472
gmrnlg1971 wants to merge 1 commit into
zaproxy:mainfrom
gmrnlg1971:fix-issue-8056

Conversation

@gmrnlg1971

@gmrnlg1971 gmrnlg1971 commented Jun 22, 2026

Copy link
Copy Markdown

Description

As part of resolving #8056, this PR adds a validation loop to DefaultVulnerabilitiesUnitTest in zap-extensions to strictly assert that getSolution() is not empty for any loaded vulnerability.

Testing

  • Unit tests pass, proving that all currently active vulnerabilities have solutions.

@github-actions

Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


Posted by the CLA Assistant Lite bot.

@psiinon

psiinon commented Jun 23, 2026

Copy link
Copy Markdown
Member

Logo
Checkmarx One – Scan Summary & Detailsaee38b0a-89b0-4037-b55b-1ab4ab97776d

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

@thc202

thc202 commented Jun 23, 2026

Copy link
Copy Markdown
Member

This is a duplicate of #7467.

@thc202 thc202 closed this Jun 23, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants