Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Introduce 3D Secure support for credit cards #232

Merged
merged 4 commits into from
Oct 26, 2019

Conversation

cedum
Copy link
Contributor

@cedum cedum commented Sep 11, 2019

This introduces 3D Secure support to satisfy the Strong Customer Authentication (SCA) requirements introduced by PSD2.

By default, 3DS is disabled. Once enabled on Braintree, it must be enabled also from Solidus Admin by checking the newly introduced checkbox in the Braintree Configuration page (see screenshot).

When enabled, here's how the checkout payment step looks:


Related issue: #162

Refs:

@cedum cedum force-pushed the 3ds-support branch 5 times, most recently from cb4512e to 312a2ea Compare September 18, 2019 12:28
@cedum cedum marked this pull request as ready for review September 18, 2019 12:31
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

cedum added 3 commits October 11, 2019 10:35
The 3.52.0 version is the minimum version required to be able to
introduce the 3D Secure 2 support.
@cedum
Copy link
Contributor Author

cedum commented Oct 11, 2019

rebased from master and force pushed

@kennyadsl
Copy link
Member

I'm trying to figure out build failures. I think that it would be helpful to also update the README adding a bit of documentation for this option, or at least, specify that we support 3D Secure on this Payment Gateway.

@cedum
Copy link
Contributor Author

cedum commented Oct 18, 2019

Added a bit of documentation on how to enable 3DS. Here's a preview: https://github.com/nebulab/solidus_paypal_braintree/tree/3ds-support#3d-secure

@kennyadsl it seems the failing specs are related to sprockets v4. We may want to fix it in the Gemfile also here to v3, wdyt?

@kennyadsl
Copy link
Member

Re-running specs, they should be ok now!

Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

👍

@kennyadsl kennyadsl merged commit e85a085 into solidusio:master Oct 26, 2019
@kennyadsl kennyadsl deleted the 3ds-support branch October 26, 2019 13:14
@delphaber
Copy link
Contributor

awesome!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants