Skip to content

ascanrules: status code heuristic for sqli to reduce false positives #5976

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

ascanrules: status code heuristic for sqli to reduce false positives #5976

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
1919a22
unit tests for error and union based sql injection
FiveOFive Oct 30, 2024
366804e
SqlInjectionScanRule unit tests for boolean based
FiveOFive Oct 4, 2024
c68f36c
migrate sqli expression + boolean checks to ComparableResponse
FiveOFive Dec 2, 2024
b4b829c
Merge branch 'sqli-boolean-based-unit-tests' into comparable-response…
FiveOFive Dec 2, 2024
ce4edbd
fix tests
FiveOFive Dec 2, 2024
7bc15b7
Fix sqli fp when confirm request gets a different status response
FiveOFive Dec 3, 2024
bd4a643
changelog
FiveOFive Dec 3, 2024
b2fd991
fix spotless
FiveOFive Dec 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions addOns/ascanrules/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
- Hidden File Finder
- User Agent Fuzzer
- Now depends on minimum Common Library version 1.29.0.
- Include status code in sql injection response comparison (Issue 8652).

### Added
- Standardized Scan Policy related alert tags on the rule.
Expand Down
Loading
Loading