ssvlabs / ssv-spec

GNU General Public License v3.0

25 stars 22 forks source link

Take the OperatorID field out of partialSigMessage #422

Open GalRogozinski opened 4 months ago

GalRogozinski commented 4 months ago

Description

It seems that the OperatorID (Signer) usage is used in PartialSigContainer and PreconsensusJustifications (currently disabled). For both cases using ValidatorIndex can work and seems to be more correct.. Once this is changed the OperatorID field can be taken out of PartialSigMessage.

GalRogozinski commented 4 months ago

Apparently I was a little wrong here. A <validatorIndex, signingRoot> pair isn't enough because each validator is separated to different parts.

However since SignedSSVMessage has OperatorIDs field we have some duplication. It is just a bit not nice that a field that is in the outer wrapper that is supposed to be used by MessageValidation will also be used for the MPC part of the protocol...