TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-11-13 01:17:27 +03:00
Code Issues Projects Releases Wiki Activity

Responded with 409 when we have DB conflicts

Browse Source 
refs https://github.com/TryGhost/Team/issues/789

We are still having issues with duplicate subscriptions being inserted,
despite running our code in transactions. For now we will catch these
errors and response ot Stripe with a 409 so that it'll retry later - and
it stops us from throwing 500's
This commit is contained in:
Fabien egg O'Carroll 2021-12-01 20:44:21 +02:00
parent 80a0e56d36
commit c99ebe589d
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
ghost/members-api/lib/services/stripe-webhook.js
View File

@ -134,7 +134,17 @@ module.exports = class StripeWebhookService {
return;
}
await this[this.handlers[event.type]](event.data.object);
try {
await this[this.handlers[event.type]](event.data.object);
} catch (err) {
if (err.code !== 'ER_DUP_ENTRY' && err.code !== 'SQLITE_CONSTRAINT') {
throw err;
}
throw new errors.GhostError({
err,
statusCode: 409
});
}
}
async subscriptionEvent(subscription) {