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

Incorrect payment method ID set on subscriptions if processing UPE redirect fails #3069

Open
james-allan opened this issue Apr 11, 2024 · 2 comments · May be fixed by #3079
Open

Incorrect payment method ID set on subscriptions if processing UPE redirect fails #3069

james-allan opened this issue Apr 11, 2024 · 2 comments · May be fixed by #3079
Assignees

Comments

@james-allan
Copy link
Contributor

Describe the bug

If the UPE redirect fails to process the payment, subscriptions purchased in the order can have the incorrect renewal processing payment method IDs stored. eg iDEAL rather than SEPA.

To Reproduce

Note: I'm not sure how to reproduce this consistantly.

Steps to reproduce the behavior:

  1. Disable Legacy UPE if you have it enabled.
  2. Enable the iDEAL payment method. You'll need EUR as your store currency and an eligible Stripe Account.
  3. Add a subscription product to your cart.
  4. Select iDEAL on checkout.
  5. When redirected to the bank site leave that tab open.
  6. View the subscriptions in your Admin dashboard and you'll notice they are set to iDEAL

Screenshot 2024-04-11 at 4 08 43 pm

  1. On Aashik's site (as mentioned here p1712809597613569/1711687628.010049-slack-C7U3Y3VMY) completing the payment at Stripe and being redirect back to the site is resulting in a nonce error. See below.

Screenshot 2024-04-11 at 4 08 43 pm

  1. The failure to process the payment redirect leads to the subscription remaining set to iDEAL rather than SEPA.

I assume the webhook comes in to successfully process the payment but it too doesn't update the subscriptions payment method.

Leaving a subscription as iDEAL will fail renewals as iDEAL cannot be used to process recurring payments.

Expected behavior

The subscription should be originally set to SEPA rather than iDEAL or the webhook processing of the should update it too.

@AashikP
Copy link

AashikP commented Apr 11, 2024

Customer report: 7951635-zd-a8c

I can replicate this consistently on my test site, but not on a new JN site.

Error after Accepting payment, and redirecting to site:

Screenshot 2024-04-11 at 3 32 46 pm

Though we see the error, the order is correctly set to Processing, and Subscription is set to active

Subscription:

image

Order:

image
  • There is no option to process renewal for the Subscription by the admin - since Payment method is iDEAL

  • Customer can't change the payment method from My Subscriptions page

  • If you try to process renewal by manually running the scheduled renewal action for the Subscription, you see the same error as you would see if this happened automatically when it is time for renewal:

    image image image

@james-allan james-allan self-assigned this Apr 16, 2024
@AashikP
Copy link

AashikP commented Apr 16, 2024

An update from 7951635-zen

It seems that it randomly picks one order and says it is via iDeal - where it will not renew any subscription if it says iDeal. Whenever it says SEPA the renewal works fine.

image

On their site, there's no the link has expired error as we can see on my test site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants