fix(connector): [worldpay] remove threeDS data from Authorize request for NTI flows (#7097)

Author: kashif-m

crates/hyperswitch_connectors/src/connectors/worldpay/requests.rs

@@ -32,7 +32,7 @@ pub struct Instruction {
     pub value: PaymentValue,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub debt_repayment: Option<bool>,
-    #[serde(rename = "threeDS")]
+    #[serde(rename = "threeDS", skip_serializing_if = "Option::is_none")]
     pub three_ds: Option<ThreeDSRequest>,
     /// For setting up mandates
     pub token_creation: Option<TokenCreation>,

crates/hyperswitch_connectors/src/connectors/worldpay/transformers.rs

@@ -390,8 +390,14 @@ fn create_three_ds_request<T: WorldpayPaymentsRequestData>(
     router_data: &T,
     is_mandate_payment: bool,
 ) -> Result<Option<ThreeDSRequest>, error_stack::Report<errors::ConnectorError>> {
-    match router_data.get_auth_type() {
-        enums::AuthenticationType::ThreeDs => {
+    match (
+        router_data.get_auth_type(),
+        router_data.get_payment_method_data(),
+    ) {
+        // 3DS for NTI flow
+        (_, PaymentMethodData::CardDetailsForNetworkTransactionId(_)) => Ok(None),
+        // 3DS for regular payments
+        (enums::AuthenticationType::ThreeDs, _) => {
             let browser_info = router_data.get_browser_info().ok_or(
                 errors::ConnectorError::MissingRequiredField {
                     field_name: "browser_info",
@@ -439,6 +445,7 @@ fn create_three_ds_request<T: WorldpayPaymentsRequestData>(
                 },
             }))
         }
+        // Non 3DS
         _ => Ok(None),
     }
 }