Skip to content

Conversation

@LarsenCundric
Copy link
Contributor

@LarsenCundric LarsenCundric commented Nov 25, 2025

Summary by cubic

Fix webhook signature verification by signing the full webhook body (event.data), not just the payload. This matches the sender’s scheme and prevents false “invalid signature” errors.

  • Bug Fixes

    • Compute HMAC over the entire body; createWebhookSignature now accepts body instead of payload. If you call this directly, update your argument key.
  • Dependencies

    • Move zod to runtime dependencies in package.json and .fern/metadata.json.

Written for commit 9e50659. Summary will update automatically on new commits.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 3 files

@gregpr07 gregpr07 merged commit fe21084 into main Nov 25, 2025
5 of 7 checks passed
@gregpr07 gregpr07 deleted the larsen/fix-webhook-encryption branch November 25, 2025 03:32
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.

3 participants