Fix Google Pay 3DS modal (#4528)

* Extract redirect URL from payment_details

* Add a unit test

* Add changelog entry

* Update changelog entry

* Update changelog entry

Co-authored-by: Mayisha <[email protected]>

* Update changelog entry

Co-authored-by: Mayisha <[email protected]>

---------

Co-authored-by: Mayisha <[email protected]>

Author: malithsen

changelog.txt

@@ -18,6 +18,7 @@
 * Update - Improve Stripe API connector logging to include request/response context
 * Fix - Fix fatal when processing setup intents for free subscriptions via webhooks
 * Fix - Prevent Stripe API calls after several consecutive 401 (Unauthorized) responses
+* Fix - 3DS authentication modal not shown when using Google Pay
 * Update - Improve Stripe API connector logging to include request/response context
 
 = 9.7.0 - 2025-07-21 =

client/express-checkout/__tests__/event-handler.test.js

@@ -616,5 +616,55 @@ describe( 'Express checkout event handlers', () => {
 			);
 			expect( completePayment ).not.toHaveBeenCalled();
 		} );
+
+		test( 'should extract redirect URL from payment_details when redirect_url is empty for 3DS authentication', async () => {
+			const threeDSRedirectUrl =
+				'#confirm-pi-pi_1234567890abcdef_secret_test1234567890abcdef:fake_nonce';
+
+			elements.submit.mockResolvedValue( {} );
+			stripe.createPaymentMethod.mockResolvedValue( {
+				paymentMethod: { id: 'pm_123' },
+			} );
+			api.expressCheckoutECECreateOrder.mockResolvedValue( {
+				payment_result: {
+					payment_status: 'success',
+					payment_details: [
+						{
+							key: 'result',
+							value: 'success',
+						},
+						{
+							key: 'redirect',
+							value: threeDSRedirectUrl,
+						},
+						{
+							key: 'payment_method',
+							value: 'pm_test1234567890abcdef',
+						},
+					],
+					redirect_url: '',
+				},
+			} );
+
+			api.confirmIntent.mockReturnValue( true );
+
+			await onConfirmHandler( {
+				api,
+				stripe,
+				elements,
+				completePayment,
+				abortPayment,
+				event,
+			} );
+
+			expect( api.expressCheckoutECECreateOrder ).toHaveBeenCalled();
+			expect( api.confirmIntent ).toHaveBeenCalledWith(
+				threeDSRedirectUrl
+			);
+			expect( completePayment ).toHaveBeenCalledWith(
+				threeDSRedirectUrl
+			);
+			expect( abortPayment ).not.toHaveBeenCalled();
+		} );
 	} );
 } );

client/express-checkout/payment-flow.js

@@ -184,11 +184,20 @@ const processOrder = async ( {
 		);
 	}
 
+	// Extract redirect URL from payment_details if redirect_url is empty
+	let redirectUrl = orderResponse?.payment_result?.redirect_url;
+	if ( ! redirectUrl ) {
+		const redirectDetail = orderResponse?.payment_result?.payment_details?.find(
+			( detail ) => detail.key === 'redirect'
+		);
+		redirectUrl = redirectDetail?.value || '';
+	}
+
 	return {
 		result: orderResponse?.payment_result?.payment_status,
 		errorMessage: orderResponse?.payment_result?.payment_details?.find(
 			( detail ) => detail.key === 'errorMessage'
 		)?.value,
-		redirect: orderResponse?.payment_result?.redirect_url,
+		redirect: redirectUrl,
 	};
 };

readme.txt

@@ -127,6 +127,7 @@ If you get stuck, you can ask for help in the [Plugin Forum](https://wordpress.o
 * Fix - Fix required field error message and PHP warning for custom checkout fields that don't have a label
 * Fix - Fix fatal when processing setup intents for free subscriptions via webhooks
 * Fix - Prevent Stripe API calls after several consecutive 401 (Unauthorized) responses
+* Fix - 3DS authentication modal not shown when using Google Pay
 * Update - Improve Stripe API connector logging to include request/response context
 
 [See changelog for full details across versions](https://raw.githubusercontent.com/woocommerce/woocommerce-gateway-stripe/trunk/changelog.txt).