We've decided to remove the option to allow all deprecated features,
because this has the effect that, if a user enables this flag, they
won't get the warning (and hence may forget to take action) at the time
that a feature is moved from the default-allowed set to the
default-denied set.
Co-authored-by: str4d <thestr4d@gmail.com>
This adds an `-allowdeprecated` CLI parameter whose value is a flag
indicating a deprecated feature that should be explicitly enabled.
Multiple instances of this argument may be provided. In the case that
this parameter is not provided, all currently deprecated RPC methods
that are not slated for removal in the next release remain available.
A user may disable all deprecated features entirely by providing the
string "none" as the argument to this parameter, or enable all
deprecated features, including those slated for removal, by providing
the string "all" as the argument to this parameter. In the case that
"all" or "none" is specified, multiple invocations of `-allowdeprecated`
are not permitted.
To explicitly enable only a specific set of deprecated features, use
`-allowdeprecated=<flag1> -allowdeprecated=<flagN> ...` when starting
zcashd. The following flags are recognized:
- "all" - enables all deprecated features.
- "none" - disables all deprecated features.
- "legacy_privacy" - enables the use of the deprecated "legacy" privacy
policy for z_sendmany. This causes the default behavior to conform to
the `FullPrivacy` directive in all cases instead of just for
transactions involving unified addresses.
- "getnewaddress" - enables the `getnewaddress` RPC method.
- "z_getnewaddress" - enables the `z_getnewaddress` RPC method.
- "zcrawreceive" - enables the `zcrawreceive` RPC method.
- "zcrawjoinsplit" - enables the `zcrawjoinsplit` RPC method.
- "zcrawkeygen" - enables the `zcrawkeygen` RPC method.
- "addrtype" - when this option is set, the deprecated `type` attribute
is returned in addition to `pool` or `address_type` (which contain the
same information) in the results of RPC methods that return address metadata.
This commit is intended to resolve#4312.
It includes:
* a link reference to zip-0200
* EXPECTED_BLOCKS_PER_HOUR: names the value and makes explicit
probabilistic nature
* ACTIVATION_TO_DEPRECATION_BLOCKS: name an intermediate value for
readability
* enforce correct value for EXPECTED_BLOCK_PER_HOUR
Application of reviewer comments
* move #include below #ifndef/#define macro guard
https://en.wikipedia.org/wiki/Include_guard
* remove code obfuscating // 48
* update zip URL
* remove code obfuscating // 14 days
Kris Nuttycombe
Jack Grigg
Kris Nuttycombe
Sasha
Jack Grigg
Kris Nuttycombe
Daira Hopwood
Sean Bowe
therealyingtong
zancas
therealyingtong
Eirik Ogilvie-Wigley