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

Brave Wallet creation fails on Windows #5930

Closed
srirambv opened this issue Nov 30, 2016 · 7 comments
Closed

Brave Wallet creation fails on Windows #5930

srirambv opened this issue Nov 30, 2016 · 7 comments

Comments

@srirambv
Copy link
Collaborator

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
Brave Wallet creation fails on Windows

Expected behavior:
Should create brave wallet

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version:
    0.13.0 Preview 2

  • Steps to reproduce:

    1. Do a clean install of 0.13.0 preview 2
    2. Enable payments to create wallet
    3. Wait for half hour and wallet is still not created
  • Screenshot if needed:
    image

  • Any related issues:
    cc: @mrose17 @ayumi

@mrose17
Copy link
Member

mrose17 commented Nov 30, 2016

@srirambv - thanks for the report. can you run it again, but with these environment variables set?

    LEDGER_LOGGING=true LEDGER_VERBOSE=true

and then attach ledger-log.json after a few minutes of no wallet creation? thanks!

@bridiver
Copy link
Collaborator

@mrose17 is webRequest.fetch() the root of this issue?

@mrose17
Copy link
Member

mrose17 commented Nov 30, 2016

good question! i suspect so. ultimately all of the ledger client calls funnel into a routine in ledger.js called roundtrip which in turn calls js/lib/request:request which in turn calls webRequest.fetch(). since we wrote that ourselves a while back, that is where i think the problem lies.

@mrose17
Copy link
Member

mrose17 commented Dec 7, 2016

@srirambv - i am using win10 x86 in a VM running https://github.com/brave/browser-laptop/releases/download/v0.12.13dev/BraveSetup-x64.exe and can not reproduce. it may be @bridiver's patch -- 6023abd -- addressed this issue.

can you reproduce using "the latest"? thanks!

@srirambv
Copy link
Collaborator Author

srirambv commented Dec 8, 2016

@mrose17 It works fine on the 0.12.xx branch. I am not able to build branch 6023abd and verify if the patch fix is working for Chromium54

@mrose17
Copy link
Member

mrose17 commented Dec 8, 2016

@srirambv - i suggest closing the issue until it can be reproduced somewhere... (-;

@srirambv
Copy link
Collaborator Author

srirambv commented Dec 8, 2016

Closing the issue for now as unable to run from 6023abd

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.