tangle-network / cggmp-threshold-ecdsa

MPC protocols for threshold ECDSA
GNU General Public License v3.0
47 stars 10 forks source link

[TASK] Remove superfluous/extra encryption in keygen #43

Open drewstone opened 1 year ago

drewstone commented 1 year ago

Issue summary When we originally modified the Zengo mp-ecdsa, we added additional encryption to P2P messages because we were originally broadcasting/gossiping all messages over our network,

Some old commits which added this extra encryption are:

I think we should remove this and make more explicit how this type of message can be encrypted before the P2P messages hit the wire.

ivokub commented 1 year ago

Yup, in CGGMP the assumption is that the channels are authenticated between parties and broadcasts are reliable. Our approach is to use Noise protocol on a higher level which provides authenticated encryption between all parties and allows to use untrusted proxy for relaying messages. See https://github.com/mpc-sdk/framework.