Skip to content

Sentryfuse consumer - #283

Open
sharvarikhamkar1304 wants to merge 27 commits into
mainfrom
Sentryfuse-Consumer
Open

Sentryfuse consumer#283
sharvarikhamkar1304 wants to merge 27 commits into
mainfrom
Sentryfuse-Consumer

Conversation

@sharvarikhamkar1304

Copy link
Copy Markdown

No description provided.

Signed-off-by: sharvarikhamkar1304 <sharvari.khamkar@mygurukulam.co>
@sharvarikhamkar1304
sharvarikhamkar1304 force-pushed the Sentryfuse-Consumer branch 2 times, most recently from 8ca419d to 9cde846 Compare July 9, 2025 10:41
Signed-off-by: sharvarikhamkar1304 <sharvari.khamkar@mygurukulam.co>
AWS Secret Key required the value be quoted and only allowed a 20-char
gap between "aws" and the value, so plain .env-style
AWS_SECRET_ACCESS_KEY=... lines were never matched, and even quoted
values missed once combined with " = " (22+ chars, over the old gap).
Widened the gap to 40 chars and made quotes optional.

GitHub PAT only matched classic ghp_ tokens - gho_/ghu_/ghs_/ghr_ and
fine-grained github_pat_ tokens were invisible to it. Extended to
cover all of them.

Verified both patterns against Go's regexp (RE2) engine - the one
actually used by sentryfuse-consumer to compile these rules - before
and after, confirming the fix and no regressions on the existing
matches.
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