diff options
author | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-01-19 17:44:27 +0100 |
---|---|---|
committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2018-01-19 17:46:17 +0100 |
commit | f4c942e3615435c9e5d7ff0cefd935be08768d0c (patch) | |
tree | fb8d7b36d94846595de88fae91df70ec1edc8b89 /CONTRIBUTING.md | |
parent | 9a97f39afaa890caa7987c6bc001b9a66e3e74e8 (diff) | |
parent | ec527c6c88146d5b36de38a1fcebe4f6ea72bd1b (diff) |
Merge #12220: Error if relative -walletdir is specified
ec527c6 Don't allow relative -walletdir paths (Russell Yanofsky)
Pull request description:
This makes it an error to explicitly specify a non-absolute -walletdir path, and also adds a debug.log warning if a relative rather than absolute -datadir path is configured.
Specifying paths relative to the current working directory in a daemon process can be dangerous, because files can fail to be located even if the configuration doesn't change, but the daemon is started up differently.
Specifying a relative -datadir now adds a warning to the debug log. It would not be backwards-compatible to forbid relative -datadir paths entirely, and it could also be inconvenient for command line testing.
Specifying a relative -walletdir now results in a startup error. But since the -walletdir option is new in 0.16.0, there should be no compatibility issues. Another reason not to use working directory paths for -walletdir specifically is that the default -walletdir is a "wallets" subdirectory inside the datadir, so it could be surprising that setting -walletdir manually would choose a directory rooted in a completely different location.
Tree-SHA512: 67cbdae677f82487a9031c5ec96b0205a488ab08718a0f4f49365e025119f3d7f6cfc88ba1eba04c1ecd8b9561a5b2c42e2e1a267af7c08c76b83e5e361f5a31
Diffstat (limited to 'CONTRIBUTING.md')
0 files changed, 0 insertions, 0 deletions