Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adyen: Correctly parse error messages for 3DS2 auths #3619

Conversation

britth
Copy link
Contributor

@britth britth commented May 6, 2020

When 3DS2 fails, Adyen will return an error under refusalReason.
Previously, we were checking for this value with the authorise
and the authorise3d endpoints, but not for authorise3ds2.
Updates message_from to properly check for the refusal reason when
present for 3DS2 txns.

Unit:
66 tests, 315 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
89 tests, 343 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

@britth britth requested a review from a team May 6, 2020 19:46
Copy link
Contributor

@curiousepic curiousepic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍
NBD but consider shortening conditional to something like %w(authorise authorise3d authorise3ds2).include?(action.to_s)

@britth britth force-pushed the ecs-1221-adyen-properly-parse-3ds2-message branch from d2e3e93 to ab5f939 Compare May 7, 2020 18:13
When 3DS2 fails, Adyen will return an error under refusalReason.
Previously, we were checking for this value with the `authorise`
and the `authorise3d` endpoints, but not for `authorise3ds2`.
Updates message_from to properly check for the refusal reason when
present for 3DS2 txns.

Unit:
66 tests, 315 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
89 tests, 343 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
@britth britth force-pushed the ecs-1221-adyen-properly-parse-3ds2-message branch from ab5f939 to 8b462de Compare May 7, 2020 18:14
@britth britth merged commit 8b462de into activemerchant:master May 7, 2020
@britth britth deleted the ecs-1221-adyen-properly-parse-3ds2-message branch May 7, 2020 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants