Skip to content
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

Wallet starts rescanning over and over #76

Closed
pacu opened this issue May 31, 2022 · 8 comments
Closed

Wallet starts rescanning over and over #76

pacu opened this issue May 31, 2022 · 8 comments

Comments

@pacu
Copy link

pacu commented May 31, 2022

Reproducing Steps:

  • Tap drawer menu button.
  • tap rescan wallet

expected: wallet rescans from 0% to completion and lets the user operate.
result: wallets rescans from 0% to 99.9% and starts over rescanning

Version: v1.7.9
lightwalletd version ECC Lightwalletd 0.4.12

@adityapk00
Copy link
Contributor

Fixed in 7f16b0f, doing a new release.

@r3ld3v
Copy link

r3ld3v commented Jun 1, 2022

@adityapk00 I see the update in the iOS App Store to 1.7.12 about 14 hours ago. I had version 1.7.9 installed and when I deleted and reinstalled the app and restored my wallet, the app itself reports 1.7.9. The wallet still just repeatedly syncs.

@zancas
Copy link

zancas commented Jun 1, 2022

@steven-ecc bullet point one of this documents an example of what we believe is the underlying issue.

We have a working version of the zecwallet-lite desktop application that resolves the issue by depending on upstream librustzcash, via our fork of zecwalletlitelib

@pacu
Copy link
Author

pacu commented Jun 1, 2022

I can report that the issue has not been fixed on 1.7.12

@pacu
Copy link
Author

pacu commented Jun 2, 2022

I just filed this issue. There might be an issue with the binary itself and the same version was uploaded instead of the one with the fixes.
#79

Credits to Steven Smith from ECC who found this issue.

@zancas
Copy link

zancas commented Jun 2, 2022

Should this issue be reopened?

@pacu
Copy link
Author

pacu commented Jun 2, 2022 via email

@zancas
Copy link

zancas commented Jun 2, 2022

We have a version of zecwalletlitelib that does not have this issue:

https://github.com/zingolabs/zecwalletlitelib/releases/tag/vzing-0.0.1

We're currently working on a version of the Android app that depends on that library, which in turn depends on:

https://github.com/zcash/librustzcash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants