Skip to content

Conversation

sahkal
Copy link
Contributor

@sahkal sahkal commented Mar 4, 2025

Type of Change

  • New feature

Description

Added ThreeDs server integration template pr

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

@sahkal sahkal added A-connector-integration Area: Connector integration S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Mar 4, 2025
@sahkal sahkal added this to the February 2025 Release milestone Mar 4, 2025
@sahkal sahkal self-assigned this Mar 4, 2025
@sahkal sahkal requested review from a team as code owners March 4, 2025 11:40
Copy link

semanticdiff-com bot commented Mar 4, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  crates/router/src/connector.rs  95% smaller
  crates/hyperswitch_connectors/src/connectors.rs  80% smaller
  crates/common_enums/src/enums.rs  62% smaller
  crates/common_enums/src/connector_enums.rs  18% smaller
  config/config.example.toml Unsupported file format
  config/deployments/integration_test.toml Unsupported file format
  config/deployments/production.toml Unsupported file format
  config/deployments/sandbox.toml Unsupported file format
  config/development.toml Unsupported file format
  config/docker_compose.toml Unsupported file format
  crates/connector_configs/src/connector.rs  0% smaller
  crates/connector_configs/toml/development.toml Unsupported file format
  crates/connector_configs/toml/sandbox.toml Unsupported file format
  crates/hyperswitch_connectors/src/connectors/juspaythreedsserver.rs  0% smaller
  crates/hyperswitch_connectors/src/connectors/juspaythreedsserver/transformers.rs  0% smaller
  crates/hyperswitch_connectors/src/default_implementations.rs  0% smaller
  crates/hyperswitch_connectors/src/default_implementations_v2.rs  0% smaller
  crates/hyperswitch_interfaces/src/configs.rs  0% smaller
  crates/router/src/core/admin.rs  0% smaller
  crates/router/src/core/payments/connector_integration_v2_impls.rs  0% smaller
  crates/router/src/core/payments/flows.rs  0% smaller
  crates/router/src/types/api.rs  0% smaller
  crates/router/src/types/api/authentication.rs  0% smaller
  crates/router/src/types/transformers.rs  0% smaller
  crates/router/tests/connectors/juspaythreedsserver.rs  0% smaller
  crates/router/tests/connectors/main.rs  0% smaller
  crates/test_utils/src/connector_auth.rs  0% smaller
  loadtest/config/development.toml Unsupported file format
  scripts/add_connector.sh Unsupported file format

swangi-kumari
swangi-kumari previously approved these changes Mar 4, 2025
ShankarSinghC
ShankarSinghC previously approved these changes Mar 5, 2025
NishantJoshi00
NishantJoshi00 previously approved these changes Mar 5, 2025
Copy link
Contributor

@NishantJoshi00 NishantJoshi00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, the comments can be addressed later on as well.

@@ -155,6 +155,7 @@ iatapay.base_url = "https://sandbox.iata-pay.iata.org/api/v1"
inespay.base_url = "https://apiflow.inespay.com/san/v21"
itaubank.base_url = "https://sandbox.devportal.itau.com.br/"
jpmorgan.base_url = "https://api-mock.payments.jpmorgan.com/api/v2"
juspaythreedsserver.base_url = "http://localhost:8000"
jpmorgan.secondary_base_url="https://id.payments.jpmorgan.com"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
jpmorgan.secondary_base_url="https://id.payments.jpmorgan.com"
jpmorgan.secondary_base_url = "https://id.payments.jpmorgan.com"

nit: consistant formatting

@@ -156,6 +156,7 @@ pub struct ConnectorTomlConfig {
#[serde_with::skip_serializing_none]
#[derive(Debug, Deserialize, serde::Serialize, Clone)]
pub struct ConnectorConfig {
pub juspaythreedsserver: Option<ConnectorTomlConfig>,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you put it in the ascending order of the names? Also, is this a single word or is it? juspay_threeds_server?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ascending order fix i will take this while completing the integration with transformers and types

@likhinbopanna likhinbopanna added this pull request to the merge queue Mar 5, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Mar 5, 2025
swangi-kumari
swangi-kumari previously approved these changes Mar 6, 2025
ShankarSinghC
ShankarSinghC previously approved these changes Mar 6, 2025
@likhinbopanna likhinbopanna added this pull request to the merge queue Mar 6, 2025
Merged via the queue into main with commit 7e5da48 Mar 6, 2025
14 of 20 checks passed
@likhinbopanna likhinbopanna deleted the 8671-integration-inrtegrate-juspay-3ds-server branch March 6, 2025 10:48
pixincreate added a commit that referenced this pull request Mar 7, 2025
* 'main' of github.com:juspay/hyperswitch:
  feat(hipay): Add Template PR (#7360)
  chore(version): 2025.03.07.0
  chore(postman): update Postman collection files
  feat(analytics): add new filters, dimensions and metrics for authentication analytics (#7451)
  feat(router): add capability to force challenge for 3DS Payments through Netcetera and send few optional fields (#7429)
  feat(analytics): refactor and rewrite authentication related analytics (#7433)
  refactor(core): Added payment id in authentication router data (#7441)
  feat(connector): Added ThreeDs server integration template pr (#7424)
  fix(xendit): Fix wasm changes (#7419)
  chore(postman): postman tests fixes (#7159)
  ci(cypress): move customer acceptance in configs to a const within commons (#7435)
  feat(core): add additional revenue recovery call flow (#7402)
  chore(version): 2025.03.06.0
  feat(payment_link): expose payment link configs for SDK UI rules and payment button (#7427)
pixincreate added a commit that referenced this pull request Mar 7, 2025
…juspay/hyperswitch into connector/gpay-mandates-authorizedotnet

* 'connector/gpay-mandates-authorizedotnet' of github.com:juspay/hyperswitch: (25 commits)
  ci(cypress): address lints and refactor redirection handler (#7437)
  chore: make v1 merchant account forward compatible (#7426)
  fix(dashboard): Added auth key to juspay threeds server (#7457)
  feat(hipay): Add Template PR (#7360)
  chore(version): 2025.03.07.0
  chore(postman): update Postman collection files
  feat(analytics): add new filters, dimensions and metrics for authentication analytics (#7451)
  feat(router): add capability to force challenge for 3DS Payments through Netcetera and send few optional fields (#7429)
  feat(analytics): refactor and rewrite authentication related analytics (#7433)
  refactor(core): Added payment id in authentication router data (#7441)
  feat(connector): Added ThreeDs server integration template pr (#7424)
  fix(xendit): Fix wasm changes (#7419)
  chore(postman): postman tests fixes (#7159)
  ci(cypress): move customer acceptance in configs to a const within commons (#7435)
  feat(core): add additional revenue recovery call flow (#7402)
  chore(version): 2025.03.06.0
  feat(payment_link): expose payment link configs for SDK UI rules and payment button (#7427)
  feat(connector): [EFT] Add EFT as a payment method (#7304)
  chore(version): 2025.03.05.1
  feat(connector): [Moneris] Implement mandate for Moneris (#7322)
  ...
@hyperswitch-bot hyperswitch-bot bot removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Mar 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-connector-integration Area: Connector integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants