Fjarri/update synedrion

[fjarri]

(had to move #525 from my fork to the main repo because CircleCI sucks)

• Fixes Move to a tagged release of synedrion, not a branch with a fix #509
• Fixes Refactor pre protocol execution  #416
• Public vs AccountId32 #376 is still open: even though we're using PublicKey instead of Public now, the problem still stands: we have AccountId32 and Public with identical contents.
• VerifierWrapper was removed, PartyId now implements the necessary traits directly.
• ProtocolMessage does not make a distinction between broadcast and direct messages anymore, since our messaging is direct-only. There is still a distinction in the protocol execution loop, but it will be removed when message bundling is implemented (see Message bundling synedrion#62)
• Removed a check for messages sent to ourselves in the protocol loop because evidently it is now checked in protocol_transport/mod.rs
• Generalized the three protocol loops.
• Note that "my_idx" is not being passed anymore from the upper levels; this information is already present in the relation between the signer and the verifiers, and if the signer's key is not present among the verifiers, there will be an error on the session creation stage (synedrion::sessions::Error::Local).

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
entropy-core	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 28, 2023 0:19am

[fjarri]

The original PR seems to have passed through CI, closing this one.