LLFourn / secp256kfun

A pure-rust secp256k1 library optimised for fun
BSD Zero Clause License
100 stars 28 forks source link

[frost] new_keygen safety #169

Closed LLFourn closed 12 months ago

LLFourn commented 12 months ago

Make it safer by encouraging passing in your secret polynomial directly.

LLFourn commented 12 months ago

Thanks @nickfarrow.

It does make new_keygen slightly clunkier, skipping the check with BTreeMap::new() or Default::default(), but i think this is probably nicer than an Option<BTreeMap<_, _>>

Clunkyness is a feature as it inspires the reader to look at the documentation!