- Issue created by @mightysooz
- Assigned to TomTech
- Status changed to Closed: works as designed
10 months ago 10:29pm 11 February 2024 - πΊπΈUnited States TomTech
Hi! Thanks for the report.
The webhook_event_id in the module code is the primary key of the commerce_stripe_webhook_event table. This is an integer.
The property you are referring to is stored in that table as the "stripe_event_id", and is stored in that table as a string.
Sounds like you might be overriding processWebhook() or onNotify()?
If you override processWebHook(), this id will already provided for you as the first argument, which you can use to update the record.
If you are overriding onNotify(), I would suggest to override processWebhook() instead, so that you can take advantage of this being provided to you. (Also, when the queued webhook event processing lands, it will be invoking processWebhook(), so it is more ideal to override.)
- π¨π¦Canada mightysooz
I am using the module out of the box, not modifying anything. I am not sure then why in this case the following webhook events would fail. For now I have disabled those events from Stripe's accounts that failed.
payment_intent.succeeded
payment_intent.created
charge.succeeded
charge.refund.updated
charge.refunded
charge.succeeded
payment_intent.succeeded
charge.refund.updated
charge.refund.updated
charge.refunded
charge.refunded
charge.refunded
payment_intent.created
charge.refund.updated - Status changed to Active
10 months ago 11:52pm 11 February 2024 - Merge request !82Issue #3420717 by mightysooz, TomTech: Webhook httpstatus 500 β (Merged) created by TomTech
- Status changed to Needs review
10 months ago 1:23am 12 February 2024 - πΊπΈUnited States TomTech
@mightysooz , thanks for the additional info.
I've created an MR to address this issue. There appears to be an issue with the phpunit tests. (Unrelated to this MR, but it is preventing the merge currently.)
In the meantime, there are two ways to work around this:
1. If you are comfortable with patching, apply the plain diff of this MR as a patch.
2. You can enable the sub-module "commerce stripe webhook events", which provides additional functionality, and doesn't have this issue. -
TomTech β
committed 276a6963 on 8.x-1.x
Issue #3420717 by mightysooz, TomTech: Webhook httpstatus 500
-
TomTech β
committed 276a6963 on 8.x-1.x
- Status changed to Fixed
9 months ago 4:28am 1 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.