Skip to content

Remove invalid characters from SendGrid API key in test data#31

Merged
azu merged 1 commit into
mainfrom
claude/fix-secretlint-1509-cDizc
May 4, 2026
Merged

Remove invalid characters from SendGrid API key in test data#31
azu merged 1 commit into
mainfrom
claude/fix-secretlint-1509-cDizc

Conversation

@azu

@azu azu commented May 4, 2026

Copy link
Copy Markdown
Member

Summary

Fixes the SendGrid API key in the demo so the rule actually detects it.

@secretlint/secretlint-rule-sendgrid enforces the official 69-character length (secretlint/secretlint#1132). The previous demo token was 75 characters (...L5528Kewhere), so the rule skipped it and the demo no longer showed a SendGrid finding. Trimming ewhere brings it to exactly 69 characters and restores detection.

Changes

  • src/DEFAUT_TEXT.ts: ...L5528Kewhere...L5528K (69 chars)

Closes secretlint/secretlint#1509

https://claude.ai/code/session_01FomGx1tfHbHaq5dT3ZLViJ

The demo SendGrid key was 75 characters and no longer matched after
@secretlint/secretlint-rule-sendgrid added strict 69-character length
validation (secretlint/secretlint#1132). Trim the trailing 'ewhere'
so the demo triggers detection again.

Refs secretlint/secretlint#1509

Co-authored-by: azu <azu@users.noreply.github.com>
@azu azu marked this pull request as ready for review May 4, 2026 09:36
@azu azu merged commit 0d7cbc5 into main May 4, 2026
1 check passed
@azu azu deleted the claude/fix-secretlint-1509-cDizc branch May 4, 2026 09:38
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.

SendGrid module not working as per demo

2 participants