IRC meeting summary for 2017-05-04
Overview
Notes / short topics
- BlueMatt brings some attention to his recent issue that questions the usefulness of the current coin control groupings for manual privacy protection.
Main topics
- HD wallet auto-restore
HD wallet auto-restore
background
Bitcoin Core has a Hierarchical Deterministic Wallet (HD wallet/BIP32) since version 0.13. A HD wallet is a system that derives all keys from a single starting point called the ‘seed’. This allows users to only back-up their wallet once and all future generated keys can be re-generated starting from the seed.
To speed up review and deployment the first version of HD wallet in Bitcoin Core was made simple without many bells and whistles. Now new features are being added to the wallet, like this HD wallet auto-restore, which ensures it always checks the keypool when synchronizing with the node and that the amount of unused keys it searches for is large enough.
meeting comments
Jonasschnelli has worked out a solution that works for encrypted (and pruned) wallets based on the conclusions of last weeks meeting, but he’s unsure what the gap limit and default keypool size should be. BlueMatt proposes a default keypool of 10k with a gap limit of 1000. Jonasschnelli notices non-encrypted wallets can stay at 100, however that would just make it more complex while key-generation is cheap.
meeting conclusion
- Bump the gap limit to 1000 and default keypool size to 10000 for all wallets.
High priority review
BlueMatt asks for #8694 (Basic multiwallet support) to be removed from the high-priority list since it needs some fixes and a rebase. Multiwallet should still be a priority for 0.15 though. One of the fixes is refactoring everything that uses mapMultiArgs, which is done in PR #9494 which should be added to the list.
Sdaftuar wants to add #9208 (Improve DisconnectTip performance)
Jtimon’s #8855 (Use a proper factory for creating chainparams) is still on the list, while easy to review.
Gmaxwell likes to see the per-txout + atomic merged soon, so there’s more testing time on the code. Sipa will do some benchmarks and report the numbers on the PR.
Participants
| IRC nick | Name/Nym |
|---|---|
| jonasschnelli | Jonas Schnelli |
| sipa | Pieter Wuille |
| cfields | Cory Fields |
| luke-jr | Luke Dashjr |
| kanzure | Bryan Bishop |
| gmaxwell | Gregory Maxwell |
| BlueMatt | Matt Corallo |
| instagibbs | Gregory Sanders |
| jtimon | Jorge Timón |
| Chris_Stewart_5 | Chris Stewart |
Disclaimer
This summary was compiled without input from any of the participants in the discussion, so any errors are the fault of the summary author and not the discussion participants.