Question about PoaNetworkConsensus contract validator states about poa-network-consensus-contracts HOT 2 OPEN

Hi @ngyam,

Yes, you're right, the isValidator shouldn't return true until the pending validator set is not finalized (the finalizeChange is called). The isValidator is lying for the newly added or removed validator in the period of time when the validator set is already changed but not yet finalized (several blocks between callings of the addValidator and finalizeChange).

We, unfortunately, missed this moment and didn't fix it when we were working on the last hard fork. However, the isValidatorFinalized getter can be used instead to determine if the specified address is a finalized validator.

For the current version of PoaNetworkConsensus contract, it's recommended to use the getValidators() getter: if some address is in the current validator set, that address will be in the returned array of the getValidators(). The getValidators() always returns an actual validator set taking into account the finalization.

Thank you for this issue - we should fix this when we have next hard fork which we'll ever do.

from poa-network-consensus-contracts.

Hi @varasev
Thanks for the clarification!

from poa-network-consensus-contracts.