develodesign / vsf-payment-stripe

Stripe payment module for Vue Storefront
MIT License
25 stars 17 forks source link

Add 3DS Auth support for Cards #50

Open Fifciu opened 3 years ago

Fifciu commented 3 years ago

Add 3DS Auth support for Cards. I've already implemented it In some project like a year ago. Here you could find my commits: https://github.com/new-fantastic/vsf-payment-stripe/commits/master

Look from 25. Feb 2020

dimasch commented 3 years ago

@Fifciu Cool feature, thanks! Can you create a pull request and i will create a next tag?

Fifciu commented 3 years ago

It will be hard for me in the current period. That's why I put a link to commits.

dimasch commented 3 years ago

@Fifciu Okay, i will do it by myself. Thanks again!

antonioglass commented 3 years ago

Hey, @dimasch! I installed this module (https://github.com/new-fantastic/vsf-payment-stripe/commits/master) but it cannot get clientSecret and there is stripeError. Have you tried this module on the latest VSF1 versions?

I also tried installing an original module but nothing happens after I place the order.

I would appreciate if you could help or update this module. Thank you!

dimasch commented 3 years ago

@antonioglass

Have you tried this module on the latest VSF1 versions?

Yes, sure, but without the 3DS feature. You can check and create a Merge Request with support this feature, welcome!

antonioglass commented 3 years ago

@dimasch Thanks for quick reply! Have you faced a similar issue like nothing happens after placing the order? There are no any errors...

dimasch commented 3 years ago

@antonioglass No, there are things that do not depend on the settings of the frontend and this module, correct configuration is still required the Magento2 module for Stripe and the api related settings (integration section)

antonioglass commented 3 years ago

Hey @Fifciu ! Does something have to be configured besides just installing your version with 3DS (https://github.com/new-fantastic/vsf-payment-stripe/commits/master)? We installed it but it works just the same way as without 3DS, so no pop up with 3DS authentication appears (of course I use a 3DS test card). Thanks!

antonioglass commented 3 years ago

Update: we looked through the code and it looks like 3DS is triggered when ClientSecret is not null but it’s all the time null. 345345345

Fifciu commented 3 years ago

So to make it work. I had to do a few changes in the core.

Many things have been done there "on the asap". So there is definitely a place for improvements. Description from above is for minimal version, but for publication it would need a refactor in my opinion.

I hope you will find my tips useful

meghasharma-24 commented 3 years ago

@Fifciu Where you added mentioned changes for comment https://github.com/develodesign/vsf-payment-stripe/issues/50#issuecomment-831832569?

Fifciu commented 3 years ago

@meghasharma-24 In the previous job for the client, I do not have access to this code anymore and not enough time to redevelop it. That's why I've prepared steps on how to do it so devs have a guide.

dearste commented 1 year ago

Any news abount this? There's a version of this module working with 3ds?

dearste commented 1 year ago

@meghasharma-24 have u implemented 3ds?