fix(core): add payment_id as query param in merchant return url

[srujanchikke]

Type of Change

• Bugfix
• New feature
• Enhancement
• Refactoring
• Dependency updates
• Documentation
• CI/CD

Description

hotfix pr for #6665

Currently in payment link after redirection in case of non 3ds, we return to merchant url with following query params

• payment id, status
  But for non 3ds we return to merchant url with following query params
• status , client secret, manual_retry_allowed

Inorder to have consistency across all return response to merchant adding payment_id to 3ds/redirection like bank redirects response url.

Additional Changes

• This PR modifies the API contract
• This PR modifies the database schema
• This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Create 3ds transaction upon returning to merchant url redirection url should have payment id .

Create 3ds failure transaction it should have the payment id in the response

Create any other redirection payment method scenarios.- tested this locally with online banking fpx for fiuu.

{
    "amount": 1000,
    "currency": "MYR",
    "amount_to_capture": 1000,
    "confirm": false,
    "payment_link" : true,
    // "profile_id" : "{{profile_id}}",
    "capture_method": "automatic",
    "payment_method": "bank_redirect",
    "payment_method_type": "online_banking_fpx",
    "return_url": "https://google.com",
    "payment_method_data": {
        "bank_redirect": {
            "online_banking_fpx": {
                "issuer": "bank_islam"
            }
        }
    }
}

Checklist

• I formatted the code cargo +nightly fmt --all
• I addressed lints thrown by cargo clippy
• I reviewed the submitted code
• I added unit tests for my changes where possible

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
crates/router/src/core/payments/helpers.rs	0% smaller