Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Transition is still occurring after update to 0.19.88 for some users #11936

Closed
LaurenWags opened this issue Nov 13, 2017 · 2 comments · Fixed by #11950
Closed

Transition is still occurring after update to 0.19.88 for some users #11936

LaurenWags opened this issue Nov 13, 2017 · 2 comments · Fixed by #11950

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Nov 13, 2017

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

@LaurenWags LaurenWags added 0.19.x issue first seen in 0.19.x bug feature/rewards priority/P2 Crashes. Loss of data. Severe memory leak. labels Nov 13, 2017
@BrendanEich
Copy link
Member

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

@bsclifton
Copy link
Member

bsclifton commented Nov 14, 2017

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 if the PIA proxy service is blocked, fallback to non-proxy server #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 😄

bsclifton added a commit that referenced this issue Nov 14, 2017
…re giving up)

Along with previous commits, this fixes #11936

Auditors: @NejcZdovc, @evq
bsclifton added a commit that referenced this issue Nov 14, 2017
…re giving up)

Along with previous commits, this fixes #11936

Auditors: @NejcZdovc, @evq

includes updated min/max values per #11950 (review)
bsclifton added a commit that referenced this issue Nov 15, 2017
…re giving up)

Along with previous commits, this fixes #11936

Auditors: @NejcZdovc, @evq

includes updated min/max values per #11950 (review)
bsclifton added a commit that referenced this issue Nov 15, 2017
…re giving up)

Along with previous commits, this fixes #11936

Auditors: @NejcZdovc, @evq

includes updated min/max values per #11950 (review)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants