diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2020-10-29 15:01:26 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2020-10-29 15:01:39 +0100 |
| commit | 42b66a6b814bca130a9ccf0a3f747cf33d628232 (patch) | |
| tree | 7423f883d4e89c093521e5a1feacec9be17d27c4 /doc | |
| parent | 8e9e190ea5ce157f1bb977cca6168e7c82275ac3 (diff) | |
| parent | 01476a88a6095fd3af71cb9bf1eadef920a1197b (diff) | |
Merge #20186: wallet: Make -wallet setting not create wallets
01476a88a6095fd3af71cb9bf1eadef920a1197b wallet: Make -wallet setting not create wallets (Russell Yanofsky)
Pull request description:
This changes `-wallet` setting to only load existing wallets, not create new ones.
- Fixes settings.json corner cases reported by sjors & promag: https://github.com/bitcoin-core/gui/issues/95, https://github.com/bitcoin/bitcoin/pull/19754#issuecomment-685858578, https://github.com/bitcoin/bitcoin/pull/19754#issuecomment-685858578
- Prevents accidental creation of wallets reported most recently by jb55 http://www.erisian.com.au/bitcoin-core-dev/log-2020-09-14.html#l-355
- Simplifies behavior after #15454. #15454 took the big step of disabling creation of the default wallet. This PR extends it to avoid creating other wallets as well. With this change, new wallets just aren't created on startup, instead of sometimes being created, sometimes not. #15454 release notes are updated here and are simpler.
This change should be targeted for 0.21.0. It's a bug fix and simplifies behavior of the #15937 / #19754 / #15454 features added in 0.21.0.
---
This PR is implementing the simplest, most basic alternative listed in https://github.com/bitcoin-core/gui/issues/95#issuecomment-694236940. Other improvements mentioned there can build on top of this.
ACKs for top commit:
achow101:
ACK 01476a88a6095fd3af71cb9bf1eadef920a1197b
hebasto:
re-ACK 01476a88a6095fd3af71cb9bf1eadef920a1197b
MarcoFalke:
review ACK 01476a88a6095fd3af71cb9bf1eadef920a1197b 🏂
Tree-SHA512: 0d50f4e5dfbd04a2efd9fd66c02085a0ed705807bdec1cf5770d0ae8cb6af07080fb81306349937bf66acdb713d03fb35636f6442b650d0820e66cbae09c2f87
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/release-notes.md | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/doc/release-notes.md b/doc/release-notes.md index 4c69c61390..d3983b1689 100644 --- a/doc/release-notes.md +++ b/doc/release-notes.md @@ -292,15 +292,18 @@ Wallet changed from `-32601` (method not found) to `-18` (wallet not found). (#20101) -### Default Wallet - -Bitcoin Core will no longer create an unnamed `""` wallet by default when no -wallet is specified on the command line or in the configuration files. For -backwards compatibility, if an unnamed `""` wallet already exists and would -have been loaded previously, then it will still be loaded. Users without an -unnamed `""` wallet and without any other wallets to be loaded on startup will -be prompted to either choose a wallet to load, or to create a new wallet. -(#15454) +### Automatic wallet creation removed + +Bitcoin Core will no longer automatically create new wallets on startup. It will +load existing wallets specified by `-wallet` options on the command line or in +`bitcoin.conf` or `settings.json` files. And by default it will also load a +top-level unnamed ("") wallet. However, if specified wallets don't exist, +Bitcoin Core will now just log warnings instead of creating new wallets with +new keys and addresses like previous releases did. + +New wallets can be created through the GUI (which has a more prominent create +wallet option), through the `bitcoin-cli createwallet` or `bitcoin-wallet +create` commands, or the `createwallet` RPC. (#15454) ### Experimental Descriptor Wallets |