rust/hyperswitch
1
0
Fork 0
mirror of https://github.com/juspay/hyperswitch.git synced 2025-11-01 02:57:02 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(router): [Trustpay] fix email & user-agent information as mandatory fields in trustpay card payment request (#1414)

Browse Source
This commit is contained in:
Arjun Karthik
2023-06-12 18:48:46 +05:30
committed by GitHub
parent 301c3dc44b
commit 7ef011ad73
2 changed files with 21 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
crates/router/src/connector/trustpay/transformers.rs
View File

@ -9,7 +9,8 @@ use serde::{Deserialize, Serialize};
use crate::{
connector::utils::{
self, AddressDetailsData, CardData, PaymentsAuthorizeRequestData, RouterData,
self, AddressDetailsData, BrowserInformationData, CardData, PaymentsAuthorizeRequestData,
RouterData,
},
consts,
core::errors,
@ -109,7 +110,7 @@ pub struct CallbackURLs {
pub error: String,
}
#[derive(Default, Debug, Serialize, PartialEq)]
#[derive(Debug, Serialize, PartialEq)]
pub struct PaymentRequestCards {
pub amount: String,
pub currency: String,
@ -130,9 +131,9 @@ pub struct PaymentRequestCards {
#[serde(rename = "billing[postcode]")]
pub billing_postcode: Secret<String>,
#[serde(rename = "customer[email]")]
pub customer_email: Option<Email>,
pub customer_email: Email,
#[serde(rename = "customer[ipAddress]")]
pub customer_ip_address: Option<std::net::IpAddr>,
pub customer_ip_address: std::net::IpAddr,
#[serde(rename = "browser[acceptHeader]")]
pub browser_accept_header: String,
#[serde(rename = "browser[language]")]
@ -220,6 +221,8 @@ fn get_card_request_data(
ccard: &api_models::payments::Card,
return_url: String,
) -> Result<TrustpayPaymentsRequest, Error> {
let email = item.request.get_email()?;
let customer_ip_address = browser_info.get_ip_address()?;
Ok(TrustpayPaymentsRequest::CardsPaymentRequest(Box::new(
PaymentRequestCards {
amount,
@ -234,8 +237,8 @@ fn get_card_request_data(
billing_country: params.billing_country,
billing_street1: params.billing_street1,
billing_postcode: params.billing_postcode,
customer_email: item.request.email.clone(),
customer_ip_address: browser_info.ip_address,
customer_email: email,
customer_ip_address,
browser_accept_header: browser_info
.accept_header
.clone()
@ -349,8 +352,16 @@ impl TryFrom<&types::PaymentsAuthorizeRouterData> for TrustpayPaymentsRequest {
screen_width: Some(1920),
time_zone: Some(3600),
accept_header: Some("*".to_string()),
user_agent: Some("none".to_string()),
ip_address: None,
user_agent: item
.request
.browser_info
.as_ref()
.and_then(|info| info.user_agent.clone()),
ip_address: item
.request
.browser_info
.as_ref()
.and_then(|info| info.ip_address),
};
let browser_info = item
.request

3
crates/router/src/connector/utils.rs
View File

@ -262,7 +262,8 @@ pub trait BrowserInformationData {
impl BrowserInformationData for types::BrowserInformation {
fn get_ip_address(&self) -> Result<std::net::IpAddr, Error> {
self.ip_address.ok_or_else(missing_field_err("ip_address"))
self.ip_address
.ok_or_else(missing_field_err("browser_info.ip_address"))
}
}