rust/hyperswitch
1
0
Fork 0
mirror of https://github.com/juspay/hyperswitch.git synced 2025-10-28 12:15:40 +08:00
Code Issues Packages Projects Releases Wiki Activity

chore: Increasing log coverage for payment method list (#5042)

Browse Source 
Co-authored-by: hyperswitch-bot[bot] <148525504+hyperswitch-bot[bot]@users.noreply.github.com>
This commit is contained in:
Prajjwal Kumar
2024-07-18 16:35:54 +05:30
committed by GitHub
parent 78a7804b9c
commit edb581e8e3
1 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
crates/router/src/core/payment_methods/cards.rs
View File

@ -2424,6 +2424,10 @@ pub async fn list_payment_methods(
.await?; .await?;
} }
} }
logger::info!(
"The Payment Methods available after Constraint Graph filtering are {:?}",
response
);
// Filter out wallet payment method from mca if customer has already saved it // Filter out wallet payment method from mca if customer has already saved it
customer customer
@ -3482,6 +3486,7 @@ pub async fn filter_payment_methods(
.unwrap_or(true); .unwrap_or(true);
let context = AnalysisContext::from_dir_values(context_values.clone()); let context = AnalysisContext::from_dir_values(context_values.clone());
logger::info!("Context created for List Payment method is {:?}", context);
let domain_ident: &[String] = &[mca_id.clone()]; let domain_ident: &[String] = &[mca_id.clone()];
let result = graph.key_value_analysis( let result = graph.key_value_analysis(
@ -3491,7 +3496,13 @@ pub async fn filter_payment_methods(
&mut cgraph::CycleCheck::new(), &mut cgraph::CycleCheck::new(),
Some(domain_ident), Some(domain_ident),
); );
if filter_pm_based_on_allowed_types if let Err(ref e) = result {
logger::error!(
"Error while performing Constraint graph's key value analysis
for list payment methods {:?}",
e
);
} else if filter_pm_based_on_allowed_types
&& filter_pm_card_network_based && filter_pm_card_network_based
&& saved_payment_methods_filter && saved_payment_methods_filter
&& matches!(result, Ok(())) && matches!(result, Ok(()))