Skip to content

Re-order payment flow when verifyBuyer is activated - #267

Merged
cesar-sosa-hol merged 2 commits into
masterfrom
DEVMOB-1149/iap-rn-plugin-2x-combined-buyer-verification-flows-verify-the-wrong
Aug 18, 2026
Merged

Re-order payment flow when verifyBuyer is activated#267
cesar-sosa-hol merged 2 commits into
masterfrom
DEVMOB-1149/iap-rn-plugin-2x-combined-buyer-verification-flows-verify-the-wrong

Conversation

@cesar-sosa-hol

Copy link
Copy Markdown
Collaborator

In 2.0–2.1.0, combined *WithBuyerVerification methods ran 3DS on a caller-supplied paymentSourceId before opening card entry / Apple Pay / Google Pay, so the verification token could belong to a different payment source than the card the buyer actually used.

This PR restores the correct 1.x / Flutter order: collect the nonce first, then verify that nonce. paymentSourceId is removed from the four combined methods (kept only on startBuyerVerificationFlow); deprecated JS overloads still accept the old argument and ignore it.

Related issues
DEVMOB-1149

Changelog
Fix combined *WithBuyerVerification methods verifying the wrong payment source. They now collect the nonce first, then verify that nonce (matching 1.x). paymentSourceId is removed from these methods; only startBuyerVerificationFlow still takes it. Deprecated JS overloads still accept the old argument and ignore it.
Document the 2.x buyer-verification APIs in docs/reference.md.
Example app no longer passes the sandbox-only ccof:customer-card-id-requires-verification value into combined buyer-verification methods.

image

@cesar-sosa-hol
cesar-sosa-hol requested a review from Armaxis August 17, 2026 19:46
Comment thread example-expo/package.json Outdated
Comment thread app.plugin.js Outdated
Comment thread android/src/main/java/com/squareinapppayments/internal/SQIPFlowLog.kt Outdated
Comment thread src/index.ts
@Armaxis
Armaxis force-pushed the DEVMOB-1149/iap-rn-plugin-2x-combined-buyer-verification-flows-verify-the-wrong branch from e6801bb to 7b6b22d Compare August 18, 2026 17:08
@cesar-sosa-hol
cesar-sosa-hol merged commit 6a68892 into master Aug 18, 2026
8 checks passed
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.

2 participants