Skip to content

fix(webhooks): verify signed inbound deliveries - #607

Merged
Oluwaseyi89 merged 3 commits into
CarbonScribe:mainfrom
rajiabdulrahmon2020-ui:fix/issue-544
Aug 30, 2026
Merged

fix(webhooks): verify signed inbound deliveries#607
Oluwaseyi89 merged 3 commits into
CarbonScribe:mainfrom
rajiabdulrahmon2020-ui:fix/issue-544

Conversation

@rajiabdulrahmon2020-ui

@rajiabdulrahmon2020-ui rajiabdulrahmon2020-ui commented Aug 29, 2026

Copy link
Copy Markdown

Submitted from rajiabdulrahmon2020-ui:fix/issue-544 and targeting the upstream repository. Please review the implementation and CI results.

Closes #544

Fixes #544

@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

@rajiabdulrahmon2020-ui Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@rajiabdulrahmon2020-ui

Copy link
Copy Markdown
Author

@CarbonScribe @rajiabdulrahmon2020-ui Status update for #544: CI has 656 passing tests and 1 failing test in the build-and-verify job; no new change was pushed because the failing test was not visible in the truncated log and needs focused diagnosis. Please review the linked CI result and rerun checks where applicable.

@rajiabdulrahmon2020-ui

Copy link
Copy Markdown
Author

@CarbonScribe @rajiabdulrahmon2020-ui PR #607 references Fixes #544. Please review/rerun CI.

@Oluwaseyi89

Copy link
Copy Markdown
Contributor

@rajiabdulrahmon2020-ui that is not how to write a descriptive PR message. You ought to explain what you have implemented, the files affected and the test written, run and passed as evidence that you implementation was correct.

@Oluwaseyi89
Oluwaseyi89 merged commit c2d8418 into CarbonScribe:main Aug 30, 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 webhook signature verification for inbound integrations

3 participants