-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
GetSignedTokens
goes into retry loop when server responds with Captcha Required Exception
#18859
Comments
@gpestana @Sneagan Is there anything else we can do to track down where this
|
@emerick Yes, I'm trying to run this down on the |
@Sneagan fixed this via https://github.com/brave/ads-serve/issues/1743, but let's keep this issue open until we address #18278. |
Closing this as #18278 is now merged. |
Verification passed on
Verified test plan from the description Verified
Logged #20131, because there is no retry logic on |
Follow up to #15600
Race condition when
GetSignedTokens
goes into retry loop when initially server responds with no captcha for wallet, So captcha is not shown whenGetSignedTokens
throws aCaptcha Required Exception
.Steps to Reproduce
https://grant.rewards.bravesoftware.com/v3/captcha/challenge/*
to respond with 404 (means no captcha)Actual result:
GetSignedTokens
goes into retry loop when server responds withCaptcha Required Exception
GetScheduledCaptcha is not re-scheduled after requests fails
Captcha is not shown
Expected result:
The captcha is shown.
Reproduces how often:
Easily reproduced
Brave version (brave://version info)
cc @emerick @tmancey @jsecretan @brave/legacy_qa @rebron
The text was updated successfully, but these errors were encountered: