feat(core): Added additional authentication fields for 3ds external authentication #8758

dgeee13 · 2025-07-25T07:36:19Z

Type of Change

Description

This PR introduces the following changes

Added additional authentication fields for 3ds external authentication, the fields are as following

challenge_code
challenge_code_reason
challenge_cancel
message_extension

Added additional fields for Cybersource payments request corresponding to the newly added auth fields
Added additional request fields for Netcetera, and other 3ds connectors corresponding to the newly added auth fields

Context - Cybersource payments via Netcetera are failing for a France based acquirer in prod, the above fields are added to fix the same.

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?

Card used for testing (mastercard with challenge flow)

5306 8899 4283 3340

Successful 3DS challenge completion flow

Tested using local hyperswitch sdk and hyperswitch backend

Test video

Connector Request for Cybersource:

cavvAlgorithm - "02"
paresStatus - "Y"
challengeCode - Null (Please note - This is not supposed to be null, checking with Netcetera team since it's coming as None from Netcetera) Fix merged in PR
paresStatusReason - Null (Please note - Null is not expected here,checking this with Netcetera team) Fix raised here PR
challengeCancelCode - Null (Please note - Null is expected here, as challenge is not cancelled)
networkScore - Null (Please note - Null is expected here, as card is not CartesBancaires and also since it is not production environment - messageExtension field is only available on production)
acsTransactionId - "9187c2cb-54f8-4e70-a63b-537046c2b34d"

3DS challenge cancel flow

Tested using local hyperswitch sdk and hyperswitch backend
Test video

Incoming Webhook from Netcetera on cancelling challenge:

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 · 2025-07-25T07:36:22Z

Review changes with

Changed Files

File	Status
crates/hyperswitch_connectors/src/connectors/nuvei/transformers.rs	15% smaller
crates/hyperswitch_connectors/src/connectors/netcetera/transformers.rs	10% smaller
crates/hyperswitch_connectors/src/connectors/cybersource/transformers.rs	10% smaller
crates/hyperswitch_domain_models/src/router_response_types.rs	1% smaller
crates/diesel_models/src/authentication.rs	0% smaller
crates/diesel_models/src/schema.rs	0% smaller
crates/diesel_models/src/schema_v2.rs	0% smaller
crates/hyperswitch_connectors/src/connectors/gpayments.rs	0% smaller
crates/hyperswitch_connectors/src/connectors/gpayments/transformers.rs	0% smaller
crates/hyperswitch_connectors/src/connectors/netcetera.rs	0% smaller
crates/hyperswitch_connectors/src/connectors/netcetera/netcetera_types.rs	0% smaller
crates/hyperswitch_connectors/src/connectors/threedsecureio.rs	0% smaller
crates/hyperswitch_connectors/src/connectors/threedsecureio/transformers.rs	0% smaller
crates/hyperswitch_connectors/src/connectors/unified_authentication_service/transformers.rs	0% smaller
crates/hyperswitch_domain_models/src/router_request_types.rs	0% smaller
crates/hyperswitch_domain_models/src/router_request_types/authentication.rs	0% smaller
crates/hyperswitch_domain_models/src/router_request_types/unified_authentication_service.rs	0% smaller
crates/hyperswitch_interfaces/src/authentication.rs	0% smaller
crates/router/src/core/authentication/utils.rs	Unsupported file format
crates/router/src/core/payments/types.rs	0% smaller
crates/router/src/core/unified_authentication_service.rs	0% smaller
crates/router/src/core/unified_authentication_service/utils.rs	Unsupported file format
crates/router/src/core/webhooks/incoming.rs	0% smaller
crates/router/src/db/authentication.rs	0% smaller
migrations/2025-07-25-115018_add_authn_fields_challenge_code_cancel_reason_message_extension/down.sql	Unsupported file format
migrations/2025-07-25-115018_add_authn_fields_challenge_code_cancel_reason_message_extension/up.sql	Unsupported file format

deepanshu-iiitu

connector changes look good

sai-harsha-vardhan · 2025-08-05T08:55:00Z

crates/hyperswitch_connectors/src/connectors/cybersource/transformers.rs

@@ -313,6 +314,7 @@ impl TryFrom<&SetupMandateRouterData> for CybersourceZeroMandateRequest {
                ))?
            }
        };
+        let cavv_algorithm = Some("2".to_string());


Could you please give some code comments explaining the reason of hardcoding this?

@sai-harsha-vardhan,
I have added a comment with the reasoning here.
Since this value is being set to "2" in multiple places, I avoided repeating the explanation throughout the code.

crates/diesel_models/src/authentication.rs

...ons/2025-07-25-115018_add_authn_fields_challenge_code_cancel_reason_message_extension/up.sql

crates/hyperswitch_connectors/src/connectors/cybersource/transformers.rs

crates/hyperswitch_connectors/src/connectors/netcetera/transformers.rs

…ordea-sepa * 'main' of github.com:juspay/hyperswitch: (89 commits) feat(router): [worldpayvantiv] add dispute list sync and implement dispute (#8830) feat(core): Add support for Void after Capture (#8839) fix(wasm): [FISERV] Added GooglePay Payment Method Type (#8832) feat(connector): [Barclaycard] Add Google Pay Payment Method (#8786) chore(version): 2025.08.06.0 feat(core): Added additional authentication fields for 3ds external authentication (#8758) refactor(core): propagate network_transaction_id in response of payment (#8829) fix(core): add fix for stopping multiple event locking idempotent logs (#8034) feat(connector): [AUTHORIZEDOTNET] create connector customer flow added (#8774) feat(core): Add L2_L3 Data Support (#8828) feat(connector): [NMI] Add mandates flow (#8652) fix(connector): [Wise] send uuid as connector_transaction_id (#8836) feat(core): populate UCS status_code in response headers (#8788) feat(external_services): Fixed Url for Unified Connector Service gRPC Client (#8587) chore: reorder v2 migrations folders (#8671) fix(router): Take merchant ID from headers in API Key - Revoke (v2) (#8808) fix(connector): (payload) currency auth key wasm changes (#8825) feat(payment-methods): add filtering logic for payment method list v2 (#8606) feat(router): add support for apple pay pre-decrypted token in the payments confirm call (#8815) chore(version): 2025.08.05.0 ...

dgeee13 self-assigned this Jul 25, 2025

dgeee13 requested review from a team as code owners July 25, 2025 07:36

dgeee13 added the A-core Area: Core flows label Jul 25, 2025

hyperswitch-bot bot added the M-database-changes Metadata: This PR involves database schema changes label Jul 25, 2025

dgeee13 force-pushed the cybersource-3ds-authentication branch 2 times, most recently from 6f623f0 to b0e017c Compare July 28, 2025 08:55

dgeee13 closed this Jul 28, 2025

dgeee13 reopened this Jul 28, 2025

dgeee13 force-pushed the cybersource-3ds-authentication branch 2 times, most recently from ebcc587 to 7cccff2 Compare July 30, 2025 14:34

dgeee13 linked an issue Jul 30, 2025 that may be closed by this pull request

[BUG] Add additional authentication fields for 3ds external authentication #8806

Closed

2 tasks

dgeee13 force-pushed the cybersource-3ds-authentication branch 3 times, most recently from 2129558 to 814560d Compare August 1, 2025 07:40

deepanshu-iiitu previously approved these changes Aug 1, 2025

View reviewed changes

dgeee13 dismissed deepanshu-iiitu’s stale review via 640dac8 August 4, 2025 11:41

dgeee13 force-pushed the cybersource-3ds-authentication branch from 640dac8 to 4e71716 Compare August 5, 2025 10:59

dgeee13 requested a review from a team as a code owner August 5, 2025 10:59

dgeee13 force-pushed the cybersource-3ds-authentication branch 2 times, most recently from fda3a7a to 01cd041 Compare August 5, 2025 11:04

dgeee13 removed the request for review from a team August 5, 2025 11:27

sai-harsha-vardhan previously approved these changes Aug 5, 2025

View reviewed changes

deepanshu-iiitu previously approved these changes Aug 5, 2025

View reviewed changes

dgeee13 requested a review from jagan-jaya August 5, 2025 12:30

jagan-jaya previously approved these changes Aug 5, 2025

View reviewed changes

SanchithHegde reviewed Aug 5, 2025

View reviewed changes

feat(core): add additional fields to authentication table

6f395bf

dgeee13 dismissed stale reviews from jagan-jaya, deepanshu-iiitu, and sai-harsha-vardhan via 6f395bf August 5, 2025 14:53

dgeee13 force-pushed the cybersource-3ds-authentication branch from 01cd041 to 6f395bf Compare August 5, 2025 14:53

Debarati Ghatak added 2 commits August 5, 2025 20:25

remove comment

f71595e

change message_extension type to pii::SecretSerdeValue

d878728

SanchithHegde reviewed Aug 5, 2025

View reviewed changes

crates/hyperswitch_connectors/src/connectors/cybersource/transformers.rs Outdated Show resolved Hide resolved

crates/hyperswitch_connectors/src/connectors/netcetera/transformers.rs Outdated Show resolved Hide resolved

add deserialization error logs

5d05ffd

dgeee13 force-pushed the cybersource-3ds-authentication branch from be506c9 to 5d05ffd Compare August 5, 2025 16:33

deepanshu-iiitu approved these changes Aug 5, 2025

View reviewed changes

SanchithHegde approved these changes Aug 5, 2025

View reviewed changes

sai-harsha-vardhan approved these changes Aug 5, 2025

View reviewed changes

likhinbopanna enabled auto-merge August 5, 2025 17:18

likhinbopanna added this pull request to the merge queue Aug 5, 2025

Merged via the queue into main with commit 89774f3 Aug 5, 2025
18 of 22 checks passed

likhinbopanna deleted the cybersource-3ds-authentication branch August 5, 2025 18:05

dgeee13 mentioned this pull request Aug 11, 2025

fix(connector): [NETCETERA] add fixes for authentication response fields #8907

Merged

14 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(core): Added additional authentication fields for 3ds external authentication #8758

feat(core): Added additional authentication fields for 3ds external authentication #8758

dgeee13 commented Jul 25, 2025 •

edited

Loading

Uh oh!

semanticdiff-com bot commented Jul 25, 2025 •

edited

Loading

Uh oh!

deepanshu-iiitu left a comment

Uh oh!

sai-harsha-vardhan Aug 5, 2025

Uh oh!

dgeee13 Aug 5, 2025 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

feat(core): Added additional authentication fields for 3ds external authentication #8758

feat(core): Added additional authentication fields for 3ds external authentication #8758

Conversation

dgeee13 commented Jul 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Type of Change

Description

Additional Changes

Motivation and Context

How did you test it?

Card used for testing (mastercard with challenge flow)

Successful 3DS challenge completion flow

3DS challenge cancel flow

Checklist

Uh oh!

semanticdiff-com bot commented Jul 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

deepanshu-iiitu left a comment

Choose a reason for hiding this comment

Uh oh!

sai-harsha-vardhan Aug 5, 2025

Choose a reason for hiding this comment

Uh oh!

dgeee13 Aug 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dgeee13 commented Jul 25, 2025 •

edited

Loading

semanticdiff-com bot commented Jul 25, 2025 •

edited

Loading

dgeee13 Aug 5, 2025 •

edited

Loading