Page MenuHomePhabricator

Do we / could we process Gr4vy transactions from gateway IPNS
Closed, ResolvedPublic

Description

Based on resolving missing transactions from PayPal recently and Adyen last year I have an unverified impression that the Gr4vy IPNs are a little less reliable than the ones direct from the Gateway. Obviously we need to get all IPNs.... so we need to work around this shortcoming. I fairly significant number seem to be missing - I *think* there have been times they have not sent them

I *think* we are getting IPNs for gravy transactions both from Gravy and direct from the processor and ignoring the direct ones. However, I also suspect it would be fairly trivial to stop ignoring the Gravy ones when they come from the processor and instead 'process whatever we get when we get it'' - we do this with the reconciliation files and it has cleared out all the missing Gravy ones & it feels like it might be a small reliability win with relatively little work (obviously for those gateways were we already have IPN handling code)

Event Timeline

Restricted Application added a subscriber: Aklapper. ยท View Herald TranscriptJan 28 2026, 7:55 PM
AKanji-WMF added a project: fr-techtalk.
AKanji-WMF subscribed.

I've added this to Tech Talk to help come to consensus as to what to do. Is there any risk to the "process whatever we get when we get it" approach?

Change #1240394 had a related patch set uploaded (by Ejegg; author: Ejegg):

[wikimedia/fundraising/SmashPig@master] WIP handle gravy messages in adyen listener

https://gerrit.wikimedia.org/r/1240394

Change #1248032 had a related patch set uploaded (by Ejegg; author: Ejegg):

[wikimedia/fundraising/SmashPig@master] WIP record gravy chargebacks from Adyen IPN listener

https://gerrit.wikimedia.org/r/1248032

Change #1248167 had a related patch set uploaded (by Ejegg; author: Ejegg):

[wikimedia/fundraising/SmashPig@master] Rename trait, drop 'Drop'

https://gerrit.wikimedia.org/r/1248167

Change #1248510 had a related patch set uploaded (by Ejegg; author: Ejegg):

[wikimedia/fundraising/SmashPig@master] Tag Adyen/Gravy messages by merchant reference

https://gerrit.wikimedia.org/r/1248510

Ejegg moved this task from Scoping/Uncommitted to Current Sprint on the Fundraising-Backlog board.
Ejegg subscribed.

Patch chain in review should at least get gravy Chargebacks into Civi via the Adyen listener.

Change #1240394 merged by jenkins-bot:

[wikimedia/fundraising/SmashPig@master] Add Gravy-related properties to AdyenMessage

https://gerrit.wikimedia.org/r/1240394

Change #1248167 merged by jenkins-bot:

[wikimedia/fundraising/SmashPig@master] Drop 'DropGravyInitiatedMessageTrait'

https://gerrit.wikimedia.org/r/1248167

Change #1248510 merged by jenkins-bot:

[wikimedia/fundraising/SmashPig@master] Tag Adyen/Gravy messages by merchant reference

https://gerrit.wikimedia.org/r/1248510

Change #1248032 merged by jenkins-bot:

[wikimedia/fundraising/SmashPig@master] Record gravy refunds/chargebacks from Adyen IPN listener

https://gerrit.wikimedia.org/r/1248032

Now happening for Adyen chargebacks at least. For example, this one is still marked as 'captured' in gravy

https://civicrm.wikimedia.org/civicrm/contact/view/contribution?reset=1&id=140086953&cid=3788252&action=view&context=contribution&selectedChild=contribute

but Civi has the correct chargeback status directly from Adyen.

XenoRyet set Final Story Points to 4.
Eileenmcnaughton renamed this task from Do we / could we process Gravy transactions from gateway IPNS to Do we / could we process Gr4vy transactions from gateway IPNS.Mon, Mar 16, 7:05 AM
Eileenmcnaughton updated the task description. (Show Details)