feat(connector): [BANKOFAMERICA] PSYNC Bugfix (#2897)

juspay · Nov 17, 2023 · bdcc138 · bdcc138
1 parent 0a88336
commit bdcc138
Showing 1 changed file with 15 additions and 6 deletions.
diff --git a/crates/router/src/connector/bankofamerica/transformers.rs b/crates/router/src/connector/bankofamerica/transformers.rs
@@ -273,15 +273,15 @@ pub enum BankofamericaPaymentStatus {
 impl ForeignFrom<(BankofamericaPaymentStatus, bool)> for enums::AttemptStatus {
     fn foreign_from((status, auto_capture): (BankofamericaPaymentStatus, bool)) -> Self {
         match status {
-            BankofamericaPaymentStatus::Authorized => {
+            BankofamericaPaymentStatus::Authorized
+            | BankofamericaPaymentStatus::AuthorizedPendingReview => {
                 if auto_capture {
                     // Because BankOfAmerica will return Payment Status as Authorized even in AutoCapture Payment
                     Self::Pending
                 } else {
                     Self::Authorized
                 }
             }
-            BankofamericaPaymentStatus::AuthorizedPendingReview => Self::Authorized,
             BankofamericaPaymentStatus::Succeeded | BankofamericaPaymentStatus::Transmitted => {
                 Self::Charged
             }
@@ -321,7 +321,7 @@ pub struct BankOfAmericaErrorInformationResponse {
 #[derive(Debug, Deserialize)]
 pub struct BankOfAmericaErrorInformation {
     reason: Option<String>,
-    message: String,
+    message: Option<String>,
 }
 
 impl<F>
@@ -369,7 +369,10 @@ impl<F>
             BankOfAmericaPaymentsResponse::ErrorInformation(error_response) => Ok(Self {
                 response: Err(types::ErrorResponse {
                     code: consts::NO_ERROR_CODE.to_string(),
-                    message: error_response.error_information.message,
+                    message: error_response
+                        .error_information
+                        .message
+                        .unwrap_or(consts::NO_ERROR_MESSAGE.to_string()),
                     reason: error_response.error_information.reason,
                     status_code: item.http_code,
                     attempt_status: None,
@@ -422,7 +425,10 @@ impl<F>
             BankOfAmericaPaymentsResponse::ErrorInformation(error_response) => Ok(Self {
                 response: Err(types::ErrorResponse {
                     code: consts::NO_ERROR_CODE.to_string(),
-                    message: error_response.error_information.message,
+                    message: error_response
+                        .error_information
+                        .message
+                        .unwrap_or(consts::NO_ERROR_MESSAGE.to_string()),
                     reason: error_response.error_information.reason,
                     status_code: item.http_code,
                     attempt_status: None,
@@ -475,7 +481,10 @@ impl<F>
             BankOfAmericaPaymentsResponse::ErrorInformation(error_response) => Ok(Self {
                 response: Err(types::ErrorResponse {
                     code: consts::NO_ERROR_CODE.to_string(),
-                    message: error_response.error_information.message,
+                    message: error_response
+                        .error_information
+                        .message
+                        .unwrap_or(consts::NO_ERROR_MESSAGE.to_string()),
                     reason: error_response.error_information.reason,
                     status_code: item.http_code,
                     attempt_status: None,