Skip to content

ascanrules: Add Example Alerts for Time Based SQLi rules - #7279

Merged
psiinon merged 1 commit into
zaproxy:mainfrom
kingthorin:mssql-ex-alerts
Apr 21, 2026
Merged

ascanrules: Add Example Alerts for Time Based SQLi rules#7279
psiinon merged 1 commit into
zaproxy:mainfrom
kingthorin:mssql-ex-alerts

Conversation

@kingthorin

Copy link
Copy Markdown
Member

Overview

  • Scan Rules > Added example alert handling.
  • Unit Tests > Added tests for the example alerts and references.
  • CHANGELOG > Added change note.

Related Issues

@psiinon

psiinon commented Apr 15, 2026

Copy link
Copy Markdown
Member

Logo
Checkmarx One – Scan Summary & Details85f3f31e-2260-472b-9d1a-5274e620a39f

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

@kingthorin
kingthorin force-pushed the mssql-ex-alerts branch 7 times, most recently from 53ea056 to a19bd8a Compare April 16, 2026 15:56
@thc202

thc202 commented Apr 16, 2026

Copy link
Copy Markdown
Member

Thank you!

@kingthorin

Copy link
Copy Markdown
Member Author

Variable name tweaked

@kingthorin

Copy link
Copy Markdown
Member Author

Needs sA, will do that shortly

@thc202

thc202 commented Apr 20, 2026

Copy link
Copy Markdown
Member

This has a conflict now.

@kingthorin

Copy link
Copy Markdown
Member Author

Addressed.

@thc202

thc202 commented Apr 20, 2026

Copy link
Copy Markdown
Member

The entries could have been merged but lgtm.

Signed-off-by: kingthorin <kingthorin@users.noreply.github.com>
@kingthorin

Copy link
Copy Markdown
Member Author

Tweaked

@psiinon
psiinon merged commit a7185b6 into zaproxy:main Apr 21, 2026
11 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 21, 2026
@kingthorin
kingthorin deleted the mssql-ex-alerts branch April 21, 2026 10:59
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