cip | title | author | discussions-to | status | type | category | created | license |
---|---|---|---|---|---|---|---|---|
53 |
Remove Minimum Client Version Check |
Karl Bartel (@karlb) |
Final |
Standards Track |
Ring 0 |
2023-02-27 |
Apache 2.0 |
Remove the minimumClientVersion
from the BlockchainParameters
contract and all code using it.
The protocol contains a mechanism to store and update a minimum client version number on the blockchain. Celo clients should periodically read this version number and shut down if they are older than the specified version. This mechanism did not prove to be useful in practice, so this CIP suggests removing it.
The client version check has been added to avoid problems when incompatible client versions interact with each other. The past has shown that changes which cause such incompatibilities always happen along with a hard fork. The hard fork ensures that incompatible versions won't be on the same fork, making additional version checks unnecessary.
Remove the following functions from the BlockchainParameters contract:
setMinimumClientVersion
getMinimumClientVersion
Celo clients will not call getMinimumClientVersion
and continue operation regardless of their version number.
There is no specific need to remove this feature, but removal will simplify the code base and reduce the divergence from upstream geth, thereby making Celo clients more maintainable. Since there are no likely scenarios in which the feature would be helpful, the maintenance costs outweigh the benefits of having the feature.
Removal of getMinimumClientVersion
in the contract breaks backwards compatibility. The client can easily be changed not to call this function, but removing the functions from the contract must wait until the next hard fork to avoid breaking existing clients.
Remove the following elements from the BlockchainParameters contract:
function setMinimumClientVersion
function getMinimumClientVersion
struct ClientVersion
ClientVersion private minimumClientVersion
- Remove
major
,minor
,patch
parameters from constructor
PR: celo-org/celo-blockchain#2026
Remove from blockchain_parameters.go
:
- SpawnCheck
- checkMinimumVersion
- The version check flag and its usage
PR: celo-org/celo-monorepo#10204
No known security considerations.
This work is licensed under the Apache License, Version 2.0.