Core Rule Set tests: name repeated attack strings (closes #263) #264

Merged
clawbot merged 1 commits from issue-263-goconst-test-strings into next 2026-11-10 01:26:08 +01:00
Collaborator

Fixes the four goconst findings that make next red. The Drupal and MediaWiki tests now use the existing constant injection for the SQL string and a new constant scriptTag for <script>alert(1)</script>. Requests and expected outcomes are unchanged.

scriptTag is not called scriptInjection because that name already holds the URL-escaped form used in query strings.

Closes #263

Model: opus-5-5

Fixes the four `goconst` findings that make `next` red. The Drupal and MediaWiki tests now use the existing constant `injection` for the SQL string and a new constant `scriptTag` for `<script>alert(1)</script>`. Requests and expected outcomes are unchanged. `scriptTag` is not called `scriptInjection` because that name already holds the URL-escaped form used in query strings. Closes https://git.eeqj.de/sneak/smallwebwaf/issues/263 Model: opus-5-5
clawbot added the needs-review label 2026-11-10 01:04:42 +01:00
clawbot self-assigned this 2026-11-10 01:04:42 +01:00
clawbot added 1 commit 2026-11-10 01:04:42 +01:00
The Drupal and MediaWiki tests repeated the SQL and script attack strings,
which goconst reports. They now use the existing constant injection and a
new constant scriptTag.

Model: opus-5-5
Author
Collaborator

PASS: the Drupal and MediaWiki tests now use injection and the new scriptTag for the repeated attack strings, with requests and expected outcomes unchanged.

Model: opus-5-5

PASS: the Drupal and MediaWiki tests now use `injection` and the new `scriptTag` for the repeated attack strings, with requests and expected outcomes unchanged. Model: opus-5-5
clawbot merged commit c1bb813a5e into next 2026-11-10 01:26:08 +01:00
clawbot deleted branch issue-263-goconst-test-strings 2026-11-10 01:26:08 +01:00
Sign in to join this conversation.