Skip to content

Add CockroachDB error-based fingerprints - #6148

Closed
sankalp73 wants to merge 1 commit into
sqlmapproject:masterfrom
sankalp73:add-cockroachdb-error-fingerprints
Closed

sankalp73 wants to merge 1 commit into
sqlmapproject:masterfrom
sankalp73:add-cockroachdb-error-fingerprints

Conversation

@sankalp73

Copy link
Copy Markdown

Fixes #6067.

Adds two passive error-message signatures for CockroachDB under the PostgreSQL block in data/xml/errors.xml: the location-first syntax error (at or near "x": syntax error, distinct from PostgreSQL's verb-first form) and the go.crdb.dev / github.com/cockroachdb hint link. Follows the existing fork-signature pattern.

Validated by feeding sample error pages through htmlParser(): CockroachDB syntax-error and hint pages resolve to PostgreSQL with the CockroachDB fork note, while genuine PostgreSQL/MySQL errors and unrelated prose are unaffected.

@stamparm

Copy link
Copy Markdown
Member

so, basically, you vibe-coded some error-parsing message, to revive the PR that i ignored for ages, most probably without even running any docker tests or anything. man, i'll need to make a NEW FAQ entry - no vibe-coding hobists allowed

@stamparm stamparm closed this Oct 18, 2026
@sankalp73
sankalp73 deleted the add-cockroachdb-error-fingerprints branch October 18, 2026 19:03
@sankalp73
sankalp73 restored the add-cockroachdb-error-fingerprints branch October 18, 2026 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants