-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add support for 3ds and surcharge decision through routing rules #2869
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add instruments to the function which does external calls like database and redis,etc.
Add instruments to the functions which invoked in payments core flows.
crates/router/src/core/payments.rs
Outdated
use super::{ | ||
errors::StorageErrorExt, | ||
payment_methods::surcharge_decision_configs::perform_surcharge_decision_management_for_session_flow, | ||
utils::persist_individual_surcharge_details_in_redis, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove here and add self in utils and use utils::fn directly, for payment_method also follow the same method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM.
@hrithikesh026 Please test the surcharge flow in this PR once
@Aprabhat19 Please test the 3DS decision manager flow in this PR once.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked the 3ds flow , its working as expected.
Tested surcharge feature and found out the surcharge was getting updated in DB from confirm request. Fixed it. Everything else works fine. |
Type of Change
Description
added support for deriving 3ds and surcharge decisions through routing rules set by merchant.
Additional Changes
Motivation and Context
How did you test it?
Steps to test : https://github.com/juspay/hyperswitch-cloud/issues/2731#issuecomment-1770147086
To test 3DS,
authentication_type
field and verify that payment goes through3ds
orno 3ds
according to the rule.Checklist
cargo +nightly fmt --all
cargo clippy