Page MenuHomePhabricator

Process the Adyen audit file like the other audit files are processed
Closed, ResolvedPublic4 Estimated Story Points

Description

Previously, we hadn't actually been using Adyen reports as audit files - just finding dangling auths and charging them. If our IPN somehow missed a notification, or if the queue was down when the donor hit our site, we'd never record the donation. We should process Adyen audit files like the others, merging data from the report with data from the logs to reconstruct any donation that may have been missed.

Event Timeline

Ejegg raised the priority of this task from to Needs Triage.
Ejegg updated the task description. (Show Details)
Ejegg added a subscriber: Ejegg.
DStrine edited a custom field.

Looks like we need to subscribe to a second report to see chargebacks. There's no type for them on the Payments Accounting Report we're using to clear uncaptured authorizations, but they are one of the journal types for Settlement Detail Report.

Ejegg removed Ejegg as the assignee of this task.Feb 4 2016, 12:07 AM

Let me know if you need any updating on current status of the processing refunds into civi (not much to say - just call mark_refund)

The current audit parser built in to SmashPig does just about nothing. It was capturing all the un-captured authorizations, and nothing else. Now that we want to leave un-captured authorizations for review, it shouldn't even do that.

Suggestion: build an audit parser in the style of AstroPay and Worldpay which runs under CRM. The initial version can satisfy this ticket by parsing out chargebacks and refunds only. I think that means we can stub out the log parsing functions. Later we can expand it to parse donation details out of logs and confirm any new donations we missed due to listener or queue outage.

Change 270353 had a related patch set uploaded (by Ejegg):
Log Adyen donation details before redirect

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

Change 270416 had a related patch set uploaded (by Ejegg):
WIP Adyen audit processor

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

Change 271450 had a related patch set uploaded (by Ejegg):
Rename get_recon_file_date function

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

Change 271450 merged by Cdentinger:
Rename get_recon_file_date function

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

Change 270353 merged by jenkins-bot:
Log Adyen donation details before redirect

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

Change 270457 had a related patch set uploaded (by Ejegg):
Adyen audit processor

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

Change 271937 had a related patch set uploaded (by Ejegg):
Enable adyen_audit in wmff build

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

Change 271938 had a related patch set uploaded (by Ejegg):
Tests for Adyen audit parser

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

Ejegg renamed this task from Process refunds in Adyen audit file to Process the Adyen audit file like the other audit files are processed.Feb 20 2016, 12:22 AM
Ejegg updated the task description. (Show Details)
Ejegg changed the point value for this task from 2 to 4.
Ejegg added a subscriber: cwdent.

Change 270457 merged by jenkins-bot:
Adyen audit processor

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

Change 270416 merged by jenkins-bot:
Adyen audit processor

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

Change 271937 merged by Ejegg:
Enable adyen_audit in wmff build

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

Change 271938 merged by jenkins-bot:
Tests for Adyen audit parser

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