Skip to content

Accept listed webhook signatures - #21

Merged
ralyodio merged 1 commit into
moshcoder:masterfrom
aiirvizionz:clientkit/webhook-signature-list
Jul 10, 2026
Merged

Accept listed webhook signatures#21
ralyodio merged 1 commit into
moshcoder:masterfrom
aiirvizionz:clientkit/webhook-signature-list

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • allow inbound webhook verification to accept space-separated Standard Webhooks signature candidates
  • keep constant-time comparison for each candidate and preserve timestamp/id validation

Why

Providers can send multiple webhook-signature values during key rotation. The previous verifier compared the whole header literally, so a valid signature was rejected when it appeared alongside another candidate.

Validation

  • npx tsc --noEmit
  • signature-list check: valid v1 candidate accepted when listed with an invalid candidate; all-invalid list rejected
  • git diff --check

@ralyodio
ralyodio merged commit 986e7bc into moshcoder:master Jul 10, 2026
2 checks passed
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