Skip to content

Add unit tests for webhook-utils signature verification - #320

Merged
Lakes41 merged 1 commit into
Adamantine-guild:mainfrom
bridgetthnkechi87-cloud:test/webhook-signature-verification
Jul 29, 2026
Merged

Add unit tests for webhook-utils signature verification#320
Lakes41 merged 1 commit into
Adamantine-guild:mainfrom
bridgetthnkechi87-cloud:test/webhook-signature-verification

Conversation

@bridgetthnkechi87-cloud

Copy link
Copy Markdown
Contributor

Closes #293

Summary

  • Added additional edge-case tests for webhook signature verification
  • Covered duplicate v1 signature values, whitespace-only signatures, byte-exact payload verification, and timestamp tolerance boundaries
  • Kept the existing Node built-in test runner
  • Did not change webhook verification source logic

Testing

  • pnpm --filter @guildpass/webhook-utils test
    • Result: 40/40 tests passed
  • pnpm --filter @guildpass/webhook-utils typecheck
    • Result: passed

Notes

  • All fixtures use fixed test-only secrets and payloads.
  • No real or production secrets are used.

@Lakes41
Lakes41 merged commit 16eb816 into Adamantine-guild:main Jul 29, 2026
1 check 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.

Add unit tests for webhook-utils signature verification

2 participants