webb-tools / cggmp-threshold-ecdsa

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

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

Open drewstone opened 10 months ago

drewstone commented 10 months 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 10 months 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.