Take the OperatorID field out of partialSigMessage
GalRogozinski opened this issue · 1 comments
GalRogozinski commented
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
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...