feat(payment_methods): add external_vault_details for payments v2 sdk session call#8003
Merged
Gnanasundari24 merged 81 commits intomainfrom Jun 6, 2025
Merged
feat(payment_methods): add external_vault_details for payments v2 sdk session call#8003Gnanasundari24 merged 81 commits intomainfrom
external_vault_details for payments v2 sdk session call#8003Gnanasundari24 merged 81 commits intomainfrom
Conversation
| None, | ||
| merchant_context.get_merchant_key_store(), | ||
| profile.get_id(), | ||
| "", // This is a placeholder for the connector name, which is not used in this context |
Contributor
There was a problem hiding this comment.
can we instead create a v2 fn with no connector_name in the input params?
su-shivanshmathur
previously approved these changes
Jun 4, 2025
sai-harsha-vardhan
approved these changes
Jun 6, 2025
| pub async fn get_merchant_connector_account_v2( | ||
| state: &SessionState, | ||
| merchant_id: &id_type::MerchantId, | ||
| creds_identifier: Option<&str>, |
Contributor
There was a problem hiding this comment.
nit: there's no creds_identitifer in V2 flows
su-shivanshmathur
approved these changes
Jun 6, 2025
AkshayaFoiger
approved these changes
Jun 6, 2025
inventvenkat
reviewed
Jun 6, 2025
| pub name: Option<Secret<String>>, | ||
| pub preprocessing_id: Option<String>, | ||
| pub payment_method_data: PaymentMethodData, | ||
| pub payment_method_data: Option<PaymentMethodData>, |
Contributor
There was a problem hiding this comment.
You are moving mandate to Optional, if it is in the response, people would have already written conditions based on this. Can you verify if there is no changes in the api contract.
Contributor
Author
There was a problem hiding this comment.
This struct is being used to propagate request data to connectors. The only place this is affecting is transformers.rs of facilitapay and the logic is added to handle in there
inventvenkat
approved these changes
Jun 6, 2025
pixincreate
added a commit
that referenced
this pull request
Jun 9, 2025
…nktransfer * 'main' of github.com:juspay/hyperswitch: (211 commits) feat(tokenio): Add OpenBanking Redirection Flow (#8152) fix: Unified scarf setup (#8238) feat(health): Health check for Decision engine (#8243) chore: Update apple pay currency filter configs (#8217) refactor(customers_v2): Remove merchant reference id from v2 customer update (#7879) chore(version): 2025.06.09.0 chore(postman): update Postman collection files ci(postman): add tunnel collection to postman tests (#8269) feat(connector): Added recurring payments support for split payments in Stripe (#8271) feat(connector): [STRIPE] Added Connector Tokenization Flow for Cards (#8248) refactor(core): introduce new field in payment_intent to handle longer return_url (#8135) fix(connector): [AUTHORIZEDOTNET] Remove country PM filters (#8278) refactor(config): redact config urls for hyperswitch vault (#8276) feat(vsaas): enable onboarding of Standard Merchant Accounts in Platform Organization (#8231) feat(router): add three_ds decision rule execute api (#8148) feat(router): Save payment method on payments confirm (V2) (#8090) ci(cypress): Add Airwallex Connector Test (#8187) feat(payment_methods): add `external_vault_details` for payments v2 sdk session call (#8003) fix(connector): [facilitapay] cpf is a required field (#8274) fix: Fixed spell check (#8227) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of Change
Description
vault_session_detailsfield to session's response for vault sdk initiationclient_secretandsession_id, this is done in sdk session call of paymentsAdditional Changes
Motivation and Context
How did you test it?
Tested through Postman:
Create an MCA with Hyperswitch Vault:
Enable vault SDK through Business Profile:
Create a Payment Intent:
Create SDK session call using the payment_id:
The response should look like below:
Checklist
cargo +nightly fmt --allcargo clippy