Browser-laptop: Transition is still occurring after update to 0.19.88 for some users

Created on 13 Nov 2017  路  2Comments  路  Source: brave/browser-laptop

Test plan

  1. Create a wallet with 0.18.36
  2. Launch into 0.19.92 or higher with LEDGER_VERBOSE=true
  3. You may see logs like ledger client is currently busy; transition will be retried shortly (this was attempt 1)
  4. If you see this and eventually the transition finishes, you know it worked 馃槃 Before, you would have seen the message ledger client is currently busy; transition will be retried on next launch

Original issue description

Description

Some users are still reporting seeing the transition after updating to 0.19.88
https://twitter.com/stshank/status/929846370835378183

Steps to Reproduce

TBD - further information needed.

Actual result:
Transition is still occuring

Expected result:
When not needed, transition should not occur.

Reproduces how often:
unknown

Brave Version

about:brave info:
0.19.88

Reproducible on current live release:
Yes

Additional Information

cc @kjozwiak

0.19.x Qchecked-Linux Qchecked-Win64 Qchecked-macOS Qtest-plan-specified bug featurrewards prioritP2 release-noteinclude

Most helpful comment

We need a timeout to stop the endless spinner in a way that the user can grok, and take better action in response to.

All 2 comments

We need a timeout to stop the endless spinner in a way that the user can grok, and take better action in response to.

OK- after some investigation, I've found at least one fatal issue which we need to handle

A different user shared logs and ran into this scenario:

  • they were a long time user (not a new user)
  • they upgraded to this new release, which has the transition to BAT
  • in the code, they qualified to transition (and call was made)

    • transition was not done because ledger client marked itself as busy

    • ledger client does not retry

The ledger client was stuck in a busy situation for this user because their workplace blocked the "privateinternetaccess.com" used for reconciling. We contact this service to get timestamps for when reconciliation needs to happen, so it's not specific to the transition

There are two actions I'm going to try and take to resolve these problems:

  1. Update the transition code to retry after a delay (15 mins, for example)
  2. Gracefully handle privateinternetaccess.com being blocked (captured with https://github.com/brave/browser-laptop/issues/11945)

An example of #2 that I initially proposed was: update the Payments screen to show fatal errors which are preventing the ledger from working (corrupt file, can't reach server, etc). @mrose17 provided a better solution which I'm incorporating now 馃槃

Was this page helpful?
0 / 5 - 0 ratings

Related issues

stevespringett picture stevespringett  路  3Comments

bbondy picture bbondy  路  3Comments

jonathansampson picture jonathansampson  路  3Comments

bsclifton picture bsclifton  路  3Comments

jonathansampson picture jonathansampson  路  3Comments