Skip to content

Conversation

prajjwalkumar17
Copy link
Member

@prajjwalkumar17 prajjwalkumar17 commented Dec 10, 2024

Type of Change

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

Description

This PR will enable JWT to be used for update config for success based routing route, previously it was only admin api key.

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?

This will be checked with JWT token.

curl --location --request PATCH '{base_url}/account/:account_id/business_profile/:profile_id/dynamic_routing/success_based/config/:routing_config_id' \
--header 'Content-Type: application/json' \
--header 'Authorization: Bearer token' \
--data '{
  "params": [
    "Currency",
    "CardBin",
    "Country",
    "PaymentMethod",
    "PaymentMethodType",
    "AuthenticationType",
    "CardNetwork"
  ]
}
'

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

Chethan-rao
Chethan-rao previously approved these changes Dec 10, 2024
Sarthak1799
Sarthak1799 previously approved these changes Dec 10, 2024
Chethan-rao
Chethan-rao previously approved these changes Dec 11, 2024
SanchithHegde
SanchithHegde previously approved these changes Dec 12, 2024
Sarthak1799
Sarthak1799 previously approved these changes Dec 12, 2024
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Dec 18, 2024
Merged via the queue into main with commit db51ec4 Dec 18, 2024
17 of 18 checks passed
@Gnanasundari24 Gnanasundari24 deleted the analytics_sr branch December 18, 2024 09:24
pixincreate added a commit that referenced this pull request Dec 20, 2024
…ete-pm

* 'main' of github.com:juspay/hyperswitch:
  refactor(dynamic_routing): add col payment_method_type in dynamic_routing_stats (#6853)
  refactor(customers_v2): include minor fixes for customer v2 flows (#6876)
  fix: cypress reports generation (#6894)
  fix(connector): Paypal BankRedirects (Ideal/EPS)  (#6864)
  chore(version): 2024.12.19.1
  fix(connector): [UNIFIED_AUTHENTICATION_SERVICE] change url path to `pre_authentication_processing` in pre-auth flow (#6885)
  refactor(users): move roles schema to global interface (#6862)
  feat(payment_methods): add support to pass apple pay recurring details to obtain apple pay merchant token (#6770)
  feat(payments): [Payment links] Add config for changing button text for payment links (#6860)
  feat(core): added customer phone_number and email to session token response for click to pay (#6863)
  FEAT(klarna): Klarna Kustom Checkout Integration (#6839)
  feat(users): handle email url for users in different tenancies (#6809)
  chore(version): 2024.12.19.0
  refactor(dynamic_routing): update the authentication for update config to include JWT type (#6785)
  chore(env): remove unified_authentication_service base_url from integ, sandbox and production toml (#6865)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-routing Area: Routing C-refactor Category: Refactor M-api-contract-changes Metadata: This PR involves API contract changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor(dynamic_routing): update the authentication for update config to include JWT type
5 participants