From b6829f36fd9e7d27814d5f719ea0b3e7202da137 Mon Sep 17 00:00:00 2001 From: Roman Khimov Date: Fri, 3 Jun 2022 11:38:03 +0300 Subject: [PATCH] config: s/HF_Aspidochelone/Aspidochelone/ HF_ prefix makes zero sense to me. If it's "hardfork", then it's in the "Hardforks" section already. If it's "hotfix", then it made some sense back when it was HF_2712_FixSyscallFees, but now it's codenamed anyway. So we can drop it and have a cleaner config. --- config/protocol.mainnet.yml | 2 +- config/protocol.testnet.yml | 2 +- config/protocol.unit_testnet.single.yml | 2 +- config/protocol.unit_testnet.yml | 2 +- docs/node-configuration.md | 2 +- pkg/config/hardfork.go | 2 +- pkg/config/hardfork_string.go | 4 ++-- pkg/config/protocol_config_test.go | 2 +- 8 files changed, 9 insertions(+), 9 deletions(-) diff --git a/config/protocol.mainnet.yml b/config/protocol.mainnet.yml index eddcc750a..3de5312c3 100644 --- a/config/protocol.mainnet.yml +++ b/config/protocol.mainnet.yml @@ -37,7 +37,7 @@ ProtocolConfiguration: VerifyTransactions: false P2PSigExtensions: false Hardforks: - HF_Aspidochelone: 1730000 + Aspidochelone: 1730000 NativeActivations: ContractManagement: [0] StdLib: [0] diff --git a/config/protocol.testnet.yml b/config/protocol.testnet.yml index 87fb8bfaf..5a086e077 100644 --- a/config/protocol.testnet.yml +++ b/config/protocol.testnet.yml @@ -40,7 +40,7 @@ ProtocolConfiguration: VerifyTransactions: false P2PSigExtensions: false Hardforks: - HF_Aspidochelone: 210000 + Aspidochelone: 210000 NativeActivations: ContractManagement: [0] StdLib: [0] diff --git a/config/protocol.unit_testnet.single.yml b/config/protocol.unit_testnet.single.yml index 7154a66a7..6bb8f6238 100644 --- a/config/protocol.unit_testnet.single.yml +++ b/config/protocol.unit_testnet.single.yml @@ -21,7 +21,7 @@ ProtocolConfiguration: OracleContract: [0] Notary: [0] Hardforks: - HF_Aspidochelone: 25 + Aspidochelone: 25 ApplicationConfiguration: # LogPath could be set up in case you need stdout logs to some proper file. diff --git a/config/protocol.unit_testnet.yml b/config/protocol.unit_testnet.yml index 320448fa5..bb3017ee7 100644 --- a/config/protocol.unit_testnet.yml +++ b/config/protocol.unit_testnet.yml @@ -30,7 +30,7 @@ ProtocolConfiguration: OracleContract: [0] Notary: [0] Hardforks: - HF_Aspidochelone: 25 + Aspidochelone: 25 ApplicationConfiguration: # LogPath could be set up in case you need stdout logs to some proper file. diff --git a/docs/node-configuration.md b/docs/node-configuration.md index 8f125869b..809eb4a2c 100644 --- a/docs/node-configuration.md +++ b/docs/node-configuration.md @@ -204,7 +204,7 @@ protocol-related settings described in the table below. | --- | --- | --- | --- | --- | | CommitteeHistory | map[uint32]int | none | Number of committee members after the given height, for example `{0: 1, 20: 4}` sets up a chain with one committee member since the genesis and then changes the setting to 4 committee members at the height of 20. `StandbyCommittee` committee setting must have the number of keys equal or exceeding the highest value in this option. Blocks numbers where the change happens must be divisible by the old and by the new values simultaneously. If not set, committee size is derived from the `StandbyCommittee` setting and never changes. | | GarbageCollectionPeriod | `uint32` | 10000 | Controls MPT garbage collection interval (in blocks) for configurations with `RemoveUntraceableBlocks` enabled and `KeepOnlyLatestState` disabled. In this mode the node stores a number of MPT trees (corresponding to `MaxTraceableBlocks` and `StateSyncInterval`), but the DB needs to be clean from old entries from time to time. Doing it too often will cause too much processing overhead, doing it too rarely will leave more useless data in the DB. | -| Hardforks | `map[string]uint32` | [] | The set of incompatible changes that affect node behaviour starting from the specified height. The default value is an empty set which should be interpreted as "each known hard-fork is applied from the zero blockchain height". The list of valid hard-fork names:
• `HF_Aspidochelone` represents hard-fork introduced in [#2469](https://github.com/nspcc-dev/neo-go/pull/2469) (ported from the [reference](https://github.com/neo-project/neo/pull/2712)). It adjusts the prices of `System.Contract.CreateStandardAccount` and `System.Contract.CreateMultisigAccount` interops so that the resulting prices are in accordance with `sha256` method of native `CryptoLib` contract. `HF_Aspidochelone` is also includes [#2519](https://github.com/nspcc-dev/neo-go/pull/2519) (ported from the [reference](https://github.com/neo-project/neo/pull/2749)). It adjusts the price of `System.Runtime.GetRandom` interop and fixes its vulnerability. | +| Hardforks | `map[string]uint32` | [] | The set of incompatible changes that affect node behaviour starting from the specified height. The default value is an empty set which should be interpreted as "each known hard-fork is applied from the zero blockchain height". The list of valid hard-fork names:
• `Aspidochelone` represents hard-fork introduced in [#2469](https://github.com/nspcc-dev/neo-go/pull/2469) (ported from the [reference](https://github.com/neo-project/neo/pull/2712)). It adjusts the prices of `System.Contract.CreateStandardAccount` and `System.Contract.CreateMultisigAccount` interops so that the resulting prices are in accordance with `sha256` method of native `CryptoLib` contract. `Aspidochelone` is also includes [#2519](https://github.com/nspcc-dev/neo-go/pull/2519) (ported from the [reference](https://github.com/neo-project/neo/pull/2749)). It adjusts the price of `System.Runtime.GetRandom` interop and fixes its vulnerability. | | KeepOnlyLatestState | `bool` | `false` | Specifies if MPT should only store latest state. If true, DB size will be smaller, but older roots won't be accessible. This value should remain th e same for the same database. | Conflicts with `P2PStateExchangeExtensions`. | | Magic | `uint32` | `0` | Magic number which uniquely identifies NEO network. | diff --git a/pkg/config/hardfork.go b/pkg/config/hardfork.go index facef3c49..5bd775122 100644 --- a/pkg/config/hardfork.go +++ b/pkg/config/hardfork.go @@ -9,7 +9,7 @@ const ( // HFAspidochelone represents hard-fork introduced in #2469 (ported from // https://github.com/neo-project/neo/pull/2712) and #2519 (ported from // https://github.com/neo-project/neo/pull/2749). - HFAspidochelone Hardfork = 1 << iota // HF_Aspidochelone + HFAspidochelone Hardfork = 1 << iota // Aspidochelone ) // hardforks holds a map of Hardfork string representation to its type. diff --git a/pkg/config/hardfork_string.go b/pkg/config/hardfork_string.go index 5778b8ed5..61aded842 100644 --- a/pkg/config/hardfork_string.go +++ b/pkg/config/hardfork_string.go @@ -11,9 +11,9 @@ func _() { _ = x[HFAspidochelone-1] } -const _Hardfork_name = "HF_Aspidochelone" +const _Hardfork_name = "Aspidochelone" -var _Hardfork_index = [...]uint8{0, 16} +var _Hardfork_index = [...]uint8{0, 13} func (i Hardfork) String() string { i -= 1 diff --git a/pkg/config/protocol_config_test.go b/pkg/config/protocol_config_test.go index c88cece9d..f95898f45 100644 --- a/pkg/config/protocol_config_test.go +++ b/pkg/config/protocol_config_test.go @@ -102,7 +102,7 @@ func TestProtocolConfigurationValidation(t *testing.T) { require.Error(t, p.Validate()) p = &ProtocolConfiguration{ Hardforks: map[string]uint32{ - "HF_Unknown": 123, // Unknown hard-fork. + "Unknown": 123, // Unknown hard-fork. }, } require.Error(t, p.Validate())