update frost-rerandomized with new Randomizer generation

Author: conradoplg

Cargo.lock

@@ -31,7 +31,8 @@ pasta_curves = { version = "0.5", default-features = false }
 rand_core = { version = "0.6", default-features = false }
 serde = { version = "1", optional = true, features = ["derive"] }
 thiserror = { version = "1.0", optional = true }
-frost-rerandomized = { version = "2.0.0-rc.0", optional = true, default-features = false, features = ["serialization", "cheater-detection"] }
+# frost-rerandomized = { version = "2.0.0", optional = true, default-features = false, features = ["serialization", "cheater-detection"] }
+frost-rerandomized = { git = "https://github.com/ZcashFoundation/frost.git", rev = "a99ec6093368b39b25de3dfe0ac772777b2c51a4", optional = true, default-features = false, features = ["serialization", "cheater-detection"] }
 
 [dependencies.zeroize]
 version = "1"
@@ -50,7 +51,8 @@ rand_chacha = "0.3"
 serde_json = "1.0"
 num-bigint = "0.4.6"
 num-traits = "0.2.19"
-frost-rerandomized = { version = "2.0.0-rc.0", features = ["test-impl"] }
+# frost-rerandomized = { version = "2.0.0", features = ["test-impl"] }
+frost-rerandomized = { git = "https://github.com/ZcashFoundation/frost.git", rev = "a99ec6093368b39b25de3dfe0ac772777b2c51a4", features = ["test-impl"] }
 
 # `alloc` is only used in test code
 [dev-dependencies.pasta_curves]

Cargo.toml

@@ -343,14 +343,36 @@ pub mod round2 {
     ///
     /// Assumes the participant has already determined which nonce corresponds with
     /// the commitment that was assigned by the coordinator in the SigningPackage.
+    #[deprecated(
+        note = "switch to sign_with_randomizer_seed(), passing a seed generated with RandomizedParams::new_from_commitments()"
+    )]
     pub fn sign(
         signing_package: &SigningPackage,
         signer_nonces: &round1::SigningNonces,
         key_package: &keys::KeyPackage,
         randomizer: Randomizer,
     ) -> Result<SignatureShare, Error> {
+        #[allow(deprecated)]
         frost_rerandomized::sign(signing_package, signer_nonces, key_package, randomizer)
     }
+
+    /// Re-randomized FROST signing using the given `randomizer_seed`, which should
+    /// be sent from the Coordinator using a confidential channel.
+    ///
+    /// See [`frost::round2::sign`] for documentation on the other parameters.
+    pub fn sign_with_randomizer_seed<C: RandomizedCiphersuite>(
+        signing_package: &SigningPackage,
+        signer_nonces: &round1::SigningNonces,
+        key_package: &keys::KeyPackage,
+        randomizer_seed: &[u8],
+    ) -> Result<SignatureShare, Error> {
+        frost_rerandomized::sign_with_randomizer_seed(
+            signing_package,
+            signer_nonces,
+            key_package,
+            randomizer_seed,
+        )
+    }
 }
 
 /// A Schnorr signature on FROST(Jubjub, BLAKE2b-512).

src/frost/redjubjub.rs

@@ -326,8 +326,7 @@ pub mod keys {
 
         /// Convert the given type to make sure the group public key has an even
         /// Y coordinate. `is_even` can be specified if evenness was already
-        /// determined beforehand. Returns a boolean indicating if the original
-        /// type had an even Y, and a (possibly converted) value with even Y.
+        /// determined beforehand.
         fn into_even_y(self, is_even: Option<bool>) -> Self;
     }
 
@@ -502,14 +501,36 @@ pub mod round2 {
     ///
     /// Assumes the participant has already determined which nonce corresponds with
     /// the commitment that was assigned by the coordinator in the SigningPackage.
+    #[deprecated(
+        note = "switch to sign_with_randomizer_seed(), passing a seed generated with RandomizedParams::new_from_commitments()"
+    )]
     pub fn sign(
         signing_package: &SigningPackage,
         signer_nonces: &round1::SigningNonces,
         key_package: &keys::KeyPackage,
         randomizer: Randomizer,
     ) -> Result<SignatureShare, Error> {
+        #[allow(deprecated)]
         frost_rerandomized::sign(signing_package, signer_nonces, key_package, randomizer)
     }
+
+    /// Re-randomized FROST signing using the given `randomizer_seed`, which should
+    /// be sent from the Coordinator using a confidential channel.
+    ///
+    /// See [`frost::round2::sign`] for documentation on the other parameters.
+    pub fn sign_with_randomizer_seed<C: RandomizedCiphersuite>(
+        signing_package: &SigningPackage,
+        signer_nonces: &round1::SigningNonces,
+        key_package: &keys::KeyPackage,
+        randomizer_seed: &[u8],
+    ) -> Result<SignatureShare, Error> {
+        frost_rerandomized::sign_with_randomizer_seed(
+            signing_package,
+            signer_nonces,
+            key_package,
+            randomizer_seed,
+        )
+    }
 }
 
 /// A Schnorr signature on FROST(Pallas, BLAKE2b-512).