stripe / stripe-terminal-ios

Stripe Terminal iOS SDK
https://stripe.com/docs/terminal/sdk/ios
Other
104 stars 65 forks source link

Discover apple pay failed using v3.0.0-b7 sdk #251

Closed jonhcbs closed 9 months ago

jonhcbs commented 1 year ago

Summary

card reader is not friendly to apple pay and often fail to pay. Seems discover apple pay card always fails, but visa apple pay card can work.

can check the video link for detail: https://www.youtube.com/shorts/1gCDd16nxdY

Code to reproduce

iOS version

16+

Installation method

cocoapods

SDK version

v3.0.0-b7

Other information

firmwareVersion: 2.01.00.17-SZZZ_Prod_US_v5-480001

bric-stripe commented 1 year ago

Hi, can you confirm this is with a US account and do you have a sample PaymentIntent id to help us investigate?

jonhcbs commented 1 year ago

@bric-stripe
Totally confirmed it's a US account.

PaymentIntent id pi_3Nmt0CAPQoqa93xj3qDmXmhK or pi_3Nmt1yAPQoqa93xj08P90w4q , there is no Terminal.shared.collectPaymentMethod callback, it's wired.

we also tried sdk 2.17.1 and 2.20.0 version, it's same issue. seems it's not related to sdk version?

bric-stripe commented 1 year ago

thanks for the additional testing and providing the PaymentIntent id's 🙏

I found this is unfortunately a known issue with Reader M2 that I wasn't aware of and is being worked on internally. The fix will likely be through a config or firmware update, and so not SDK, but I'll update this issue when we know more.

jonhcbs commented 1 year ago

hello @bric-stripe, any update on this, seems it's weeks.

bric-stripe commented 1 year ago

we believe we have a root cause now and a fix is in the works. No ETA for it yet. The fix will require an SDK update.

bric-stripe commented 9 months ago

Wanted to follow up and report that although 3.3.0 fixes this issue we're actively working on a 3.3.1 to address a crash connecting to Chippers and M2 readers that are running older configs. We'll post updates in https://github.com/stripe/stripe-terminal-ios/issues/282