Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐛 Fix Documentation #467

Closed
joesaby opened this issue Mar 29, 2025 · 5 comments
Closed

🐛 Fix Documentation #467

joesaby opened this issue Mar 29, 2025 · 5 comments
Labels
bug Something isn't working

Comments

@joesaby
Copy link

joesaby commented Mar 29, 2025

Note

Please stick to the template and provide as much detail as possible to help us diagnose and fix the issue.
Low effort bug reports will be closed.

Describe the bug
Documentation says incorrect Stripe webhook link

To Reproduce
Steps to reproduce the behavior:

  1. Go to Docs link

Expected behavior
Says
"Set the webhook URL to: https://your-app.com/api/webhooks/stripe"

Should be Set the webhook URL to: https://your-app.com/public/webhooks/stripe

@joesaby joesaby added the bug Something isn't working label Mar 29, 2025
@joesaby
Copy link
Author

joesaby commented Mar 29, 2025

Could have created a PR given this is whats in code

$router->post('/public/webhooks/stripe', StripeIncomingWebhookAction::class);

@joesaby
Copy link
Author

joesaby commented Mar 29, 2025

If I use https://your-app.com/api/webhooks/stripe, I get a 404 from stripe

@daveearley
Copy link
Contributor

Hi @joesaby,

Thanks for pointing this out. I've updated the docs with the correct URL.

@joesaby
Copy link
Author

joesaby commented Mar 31, 2025

Thanks. Changing the URL, I get a different error

Why would this occur?

[2025-03-31 15:51:36] production.ERROR: Failed to handle incoming Stripe webhook {"exception":"[object] (Stripe\\Exception\\SignatureVerificationException(code: 0): No signatures found matching the expected signature for payload at /app/backend/vendor/stripe/stripe-php/lib/Exception/SignatureVerificationException.php:28)
[2025-03-31 15:54:05] production.ERROR: No signatures found matching the expected signature for payload [{"file":"/app/backend/vendor/stripe/stripe-php/lib/WebhookSignature.php","line":58,"function":"factory","class":"Stripe\\Exception\\SignatureVerificationException","type":"::"},{"file":"/app/backend/vendor/stripe/stripe-php/lib/Webhook.php","line":29,"function":"verifyHeader","class":"Stripe\\WebhookSignature","type":"::"},{"file":"/app/backend/app/Services/Application/Handlers/Order/Payment/Stripe/IncomingWebhookHandler.php","line":48,"function":"constructEvent","class":"Stripe\\Webhook","type":"::"},{"file":"laravel-serializable-closure://static function (\\HiEvents\\Services\\Application\\Handlers\\Order\\Payment\\Stripe\\IncomingWebhookHandler $handler) use ($headerSignature, $payload) {

@joesaby
Copy link
Author

joesaby commented Mar 31, 2025

NVM. The webhook key was incorrect. Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants