crypto: Add serde for BLSAggregateSignature

MystenLabs · Jan 4, 2023 · f53860f · f53860f
1 parent 4886087
commit f53860f
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/fastcrypto/src/bls12381/mod.rs b/fastcrypto/src/bls12381/mod.rs
@@ -95,22 +95,19 @@ pub struct BLS12381KeyPair {
 /// BLS 12-381 signature.
 #[readonly::make]
 #[serde_as]
-#[derive(Debug, Clone, Serialize, Deserialize)]
+#[derive(Debug, Clone)]
 pub struct BLS12381Signature {
-    #[serde_as(as = "BlsSignature")]
     pub sig: blst::Signature,
-    #[serde(skip)]
     pub bytes: OnceCell<[u8; $sig_length]>,
 }
 
+serialize_deserialize_with_to_from_bytes!(BLS12381Signature);
+
 /// Aggregation of multiple BLS 12-381 signatures.
 #[readonly::make]
-#[serde_as]
-#[derive(Debug, Clone, Serialize, Deserialize)]
+#[derive(Debug, Clone)]
 pub struct BLS12381AggregateSignature {
-    #[serde_as(as = "Option<BlsSignature>")]
     pub sig: Option<blst::Signature>,
-    #[serde(skip)]
     pub bytes: OnceCell<[u8; $sig_length]>,
 }
 
@@ -714,6 +711,8 @@ impl ToFromBytes for BLS12381AggregateSignature {
     }
 }
 
+serialize_deserialize_with_to_from_bytes!(BLS12381AggregateSignature);
+
 pub struct BlsSignature;
 
 impl SerializeAs<blst::Signature> for BlsSignature {