Skip to content

G4-E4: exclude.branches pattern uses extglob +(a|b) (disallowed chars, the original ReDoS vector) - expect SCAN - #61

Open
alan-hacktron wants to merge 1 commit into
mainfrom
g4-e4-src
Open

G4-E4: exclude.branches pattern uses extglob +(a|b) (disallowed chars, the original ReDoS vector) - expect SCAN#61
alan-hacktron wants to merge 1 commit into
mainfrom
g4-e4-src

Conversation

@alan-hacktron

Copy link
Copy Markdown
Owner

Pattern release/+(a|b) uses extglob syntax — the undocumented micromatch feature that caused the original ReDoS finding this whole limit system was built to close. ALLOWED_PATTERN_CHARS bans parens outright, which kills every extglob form. Same label=skip-scan pairing. Expected: SCAN (fail-open), plus a malformed-config bot comment.

@hacktron-app-stg

Copy link
Copy Markdown

Hacktron config could not be read

schema validation failed: exclude.branches: each pattern must be at most 100 characters with at most 5 wildcards and cannot contain advanced glob characters (like braces, brackets, parentheses, or extglobs)

Hacktron is scanning this PR using your organization's default settings instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant