-
Notifications
You must be signed in to change notification settings - Fork 879
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
Brave Ads server retry logic should backoff exponentially with a ceiling #5197
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rewards code looks good
ad7f7d3
to
b11e6e7
Compare
CI failed due to unrelated android code-signing issue |
Verification passed with Windows 10 x64 using
Catalog before changing the system time
Set the system time 2 hours ahead and confirmed that new catalog gets downloaded and the timestamp is different for both the catalogs
Verified the case by enabling rewards and making sure that the catalog file gets created in
Charles configuration: Host: Failed to refill tokens
Failed to redeem tokens
Retry Failed confirmation
After successful retry failed confirmation queue becomed empty for
Faied ads rewards and get payment balance
|
Resolves brave/brave-browser#7226
Requires #5228
Submitter Checklist:
npm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
ads-serve.brave.com/v2/catalog
for production orads-serve.bravesoftware.com/v2/catalog
for staging)GET /v1/confirmation/token/{payment_id}?nonce={nonce}
)POST /v1/confirmation/{confirmation_id}/{credential}
andGET /v1/confirmation/{confirmation_id}/paymentToken
)status code for
PUT /v1/confirmation/payment/{payment_id}
)GET /v1/confirmation/payment/{payment_id}
andGET /v2/wallet/{payment_id}/grants/ads
)Reviewer Checklist:
After-merge Checklist:
changes has landed on.