From 3c98e85c0aed5d35b20954e6c4055ded2a94546e Mon Sep 17 00:00:00 2001 From: Luke Parker Date: Sat, 23 Apr 2022 12:59:53 -0400 Subject: [PATCH] Futher document hash_msg --- sign/frost/src/lib.rs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/sign/frost/src/lib.rs b/sign/frost/src/lib.rs index f898b301..5c4c118c 100644 --- a/sign/frost/src/lib.rs +++ b/sign/frost/src/lib.rs @@ -59,6 +59,12 @@ pub trait Curve: Clone + Copy + PartialEq + Eq + Debug { /// Hash the message as needed to calculate the binding factor /// H3 from the IETF draft + // This doesn't actually need to be part of Curve as it does nothing with the curve + // This also solely relates to FROST and with a proper Algorithm/HRAM, all projects using + // aggregatable signatures over this curve will work without issue, albeit potentially with + // incompatibilities between FROST implementations + // It is kept here as Curve + HRAM is effectively a ciphersuite according to the IETF draft + // and moving it to Schnorr would force all of them into being ciphersuite-specific fn hash_msg(msg: &[u8]) -> Vec; /// Field element from hash, used in key generation and to calculate the binding factor