Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sometimes the status check QR code doesn't wait for the previous request to complete or times out. #2499

Closed
drawtsae opened this issue Dec 22, 2023 · 3 comments

Comments

@drawtsae
Copy link

drawtsae commented Dec 22, 2023

Describe the bug
Occasionally we encounter a case of drop-in trigger onAdditionalDetails 2 times in a row just a few milliseconds apart when the user scans the QR code. My system cannot handle this duplicate request, I was workaround by implement queue for this. We were able to replicate this case when the network was unstable.

To Reproduce
Steps to reproduce the behavior:

  1. Check out with payment have QR code. in this case I use Paynow, on normal network condition, even offline, it work fine.
    image

  2. But when I change Network throttling to very slow (up: 8kbit/s,down: 8kbit/s), many API check status will be call
    image

  3. At this time, if use try to scan QR code, many API check status will return sucess and trigger onAdditionalDetails

Expected behavior
Can we make sure it retry until getting a complete response from the server or it times out.

@longyulongyu
Copy link
Contributor

@drawtsae

Thanks for reporting.
Which sdk version are you integrating with? There isn't an onSubmitAdditionalDetail callback. Do you mean onAdditionalDetails callback?

@drawtsae
Copy link
Author

drawtsae commented Dec 28, 2023

@longyulongyu
@adyen/adyen-web: 5.53.2

I use version 5.53.2.
Sorry for the mistake, and yes it does trigger onAdditionalDetails more than 1 time.

@drawtsae
Copy link
Author

drawtsae commented Jan 10, 2024

Hi @longyulongyu, I see the issue has been fixed in the latest release version 5.56.0, I checked and it worked as expected. Thanks for your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants