Skip to content

Conversation

sai-harsha-vardhan
Copy link
Contributor

@sai-harsha-vardhan sai-harsha-vardhan commented Jun 28, 2024

Type of Change

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

Description

Currently few no_three_ds payments are failing with Authentication required error message on PROD. To tackle this, We are sending browser_info for all the card and googlepay payments which would allow connector to perform authentication for no_three_ds payments when required

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?

Tested Manually

  1. Sanity No3DS Payment through Adyen
"payment_method_data": {
        "card": {
            "card_number": "4111111111111111",
            "card_exp_month": "03",
            "card_exp_year": "2030",
            "card_holder_name": "John",
            "card_cvc": "737"
        }
    }

image

  1. Sanity 3DS Payment through Adyen
"payment_method_data": {
        "card": {
            "card_number": "4917610000000000",
            "card_exp_month": "03",
            "card_exp_year": "2030",
            "card_holder_name": "John",
            "card_cvc": "737"
        }
    }

image

  1. Sanity Google Pay payment though Adyen
image

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

@sai-harsha-vardhan sai-harsha-vardhan added A-connector-integration Area: Connector integration C-bug Category: Bug S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Jun 28, 2024
@sai-harsha-vardhan sai-harsha-vardhan added this to the June 2024 Release milestone Jun 28, 2024
@sai-harsha-vardhan sai-harsha-vardhan self-assigned this Jun 28, 2024
@sai-harsha-vardhan sai-harsha-vardhan requested a review from a team as a code owner June 28, 2024 06:45
@sai-harsha-vardhan sai-harsha-vardhan changed the title fix(connector): [ADYEN] send browser_info for all the card payments fix(connector): [ADYEN] send browser_info for all the card and googlepay payments Jun 28, 2024
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jun 28, 2024
Merged via the queue into main with commit a172cba Jun 28, 2024
@Gnanasundari24 Gnanasundari24 deleted the send-browser-info-for-all-card-payments-adyen branch June 28, 2024 08:32
pixincreate added a commit that referenced this pull request Jun 28, 2024
…ror-handling-in-cypress

* 'main' of github.com:juspay/hyperswitch:
  Docs: Api reference docs update for Payments - Create (#4955)
  feat(cypress): add iatapay connector (#5093)
  chore: fix ui-test configs (#5152)
  chore(cards): add configuration option to change the decryption scheme locker (#5140)
  refactor(hyperswitch_constraint_graph): Removal of lifetime from the Constraint Graph framework (#5132)
  feat(core): customer_details storage in payment_intent (#5007)
  fix(connector): [ADYEN] send `browser_info` for all the card and googlepay payments (#5153)
  fix(users): clear cookie and alter parsing for sso (#5147)
  refactor(connector): added amount framework to paypal, payouts and routing (#4865)
  chore(version): 2024.06.28.0
  chore(postman): update Postman collection files
  chore: use generic phone numbers instead (#5142)
@SanchithHegde SanchithHegde removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Jun 30, 2024
sai-harsha-vardhan added a commit that referenced this pull request Jul 2, 2024
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 C-bug Category: Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants