solidusio / solidus_stripe

💳 Integrate Solidus with Stripe
https://stripe.com
BSD 3-Clause "New" or "Revised" License
37 stars 61 forks source link

Protect against concurrent duplicated webhooks #286

Closed waiting-for-dev closed 1 year ago

waiting-for-dev commented 1 year ago

Summary

By acquiring an update lock on the payment, we can ensure that only one concurrent excecution of the webhook handler will manipulate it. That will avoid duplicated log entries and obscure exceptions.

Closes #188

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

The following are not always needed: