implemented acquireCredential method in the wasm credential client

Cargo.toml

@@ -172,6 +172,12 @@ url = "2.4"
 zeroize = "1.6.0"
 
 # wasm-related dependencies
+
+# The `console_error_panic_hook` crate provides better debugging of panics by
+# logging them with `console.error`. This is great for development, but requires
+# all the `std::fmt` and `std::panicking` infrastructure, so isn't great for
+# code size when deploying.
+console_error_panic_hook = "0.1.7"
 gloo-utils = "0.1.7"
 js-sys = "0.3.63"
 serde-wasm-bindgen = "0.5.0"

common/bandwidth-controller/src/acquire/mod.rs

@@ -19,14 +19,18 @@ use std::str::FromStr;
 
 pub mod state;
 
-pub async fn deposit<C>(client: &C, amount: Coin) -> Result<State, BandwidthControllerError>
+pub async fn deposit<C>(
+    client: &C,
+    amount: impl Into<Coin>,
+) -> Result<State, BandwidthControllerError>
 where
     C: CoconutBandwidthSigningClient + Sync,
 {
     let mut rng = OsRng;
     let signing_keypair = KeyPair::from(identity::KeyPair::new(&mut rng));
     let encryption_keypair = KeyPair::from(encryption::KeyPair::new(&mut rng));
     let params = Parameters::new(TOTAL_ATTRIBUTES).unwrap();
+    let amount = amount.into();
     let voucher_value = amount.amount.to_string();
 
     let tx_hash = client

common/client-libs/validator-client/src/nyxd/error.rs

@@ -44,7 +44,7 @@ pub enum NyxdError {
     #[error("{0} is not a valid tx hash")]
     InvalidTxHash(String),
 
-    #[error("Tendermint RPC request failed - {0}")]
+    #[error("Tendermint RPC request failed: {0}")]
     TendermintErrorRpc(#[from] TendermintRpcError),
 
     #[error("tendermint library failure: {0}")]
@@ -56,22 +56,22 @@ pub enum NyxdError {
     #[error("Failed when attempting to deserialize data ({0})")]
     DeserializationError(String),
 
-    #[error("Failed when attempting to encode our protobuf data - {0}")]
+    #[error("Failed when attempting to encode our protobuf data: {0}")]
     ProtobufEncodingError(#[from] prost::EncodeError),
 
-    #[error("Failed to decode our protobuf data - {0}")]
+    #[error("Failed to decode our protobuf data: {0}")]
     ProtobufDecodingError(#[from] prost::DecodeError),
 
-    #[error("Account {0} does not exist on the chain")]
+    #[error("Account '{0}' does not exist on the chain")]
     NonExistentAccountError(AccountId),
 
-    #[error("Failed on json serialization/deserialization - {0}")]
+    #[error("Failed on json serialization/deserialization: {0}")]
     SerdeJsonError(#[from] serde_json::Error),
 
-    #[error("Account {0} is not a valid account address")]
+    #[error("Account '{0}' is not a valid account address")]
     MalformedAccountAddress(String),
 
-    #[error("Account {0} has an invalid associated public key")]
+    #[error("Account '{0}' has an invalid associated public key")]
     InvalidPublicKey(AccountId),
 
     #[error("Queried contract (code_id: {0}) did not have any code information attached")]
@@ -86,7 +86,7 @@ pub enum NyxdError {
     #[error("Block has an invalid height (either negative or larger than i64::MAX")]
     InvalidHeight,
 
-    #[error("Failed to compress provided wasm code - {0}")]
+    #[error("Failed to compress provided wasm code: {0}")]
     WasmCompressionError(io::Error),
 
     #[error("Logs returned from the validator were malformed")]

common/client-libs/validator-client/src/rpc/reqwest.rs

@@ -108,7 +108,8 @@ trait TendermintRpcErrorMap {
 
 impl TendermintRpcErrorMap for reqwest::Error {
     fn into_rpc_err(self) -> Error {
-        todo!()
+        // that's not the best error converion, but it's better than a panic
+        Error::client_internal(self.to_string())
     }
 }
 

common/credential-storage/Cargo.toml

@@ -7,10 +7,9 @@ edition = "2021"
 
 [dependencies]
 async-trait = { workspace = true }
-
 log = { workspace = true }
-thiserror = "1.0"
-tokio = { version = "1.24.1", features = ["sync"]}
+thiserror = { workspace = true }
+tokio = { workspace = true, features = ["sync"]}
 
 [target."cfg(not(target_arch = \"wasm32\"))".dependencies.sqlx]
 version = "0.5"
@@ -23,4 +22,4 @@ features = [ "rt-multi-thread", "net", "signal", "fs" ]
 
 [build-dependencies]
 sqlx = { version = "0.5", features = ["runtime-tokio-rustls", "sqlite", "macros", "migrate"] }
-tokio = { version = "1.24.1", features = ["rt-multi-thread", "macros"] }
+tokio = { workspace = true, features = ["rt-multi-thread", "macros"] }

common/credential-storage/src/models.rs

@@ -1,7 +1,7 @@
 // Copyright 2022 - Nym Technologies SA <[email protected]>
 // SPDX-License-Identifier: Apache-2.0
 
-#[derive(Clone)]
+#[derive(Clone, Debug)]
 pub struct CoconutCredential {
     #[allow(dead_code)]
     pub id: i64,

common/credentials/src/coconut/bandwidth.rs

@@ -6,16 +6,16 @@
 // right now this has no double-spending protection, spender binding, etc
 // it's the simplest possible case
 
+use super::utils::prepare_credential_for_spending;
+use crate::error::Error;
 use cosmrs::tendermint::hash::Algorithm;
 use cosmrs::tendermint::Hash;
 use nym_coconut_interface::{
     hash_to_scalar, prepare_blind_sign, Attribute, BlindSignRequest, Credential, Parameters,
     PrivateAttribute, PublicAttribute, Signature, VerificationKey,
 };
 use nym_crypto::asymmetric::{encryption, identity};
-
-use super::utils::prepare_credential_for_spending;
-use crate::error::Error;
+use std::fmt::{self, Debug, Formatter};
 
 pub const PUBLIC_ATTRIBUTES: u32 = 2;
 pub const PRIVATE_ATTRIBUTES: u32 = 2;
@@ -44,6 +44,27 @@ pub struct BandwidthVoucher {
     blind_sign_request: BlindSignRequest,
 }
 
+impl Debug for BandwidthVoucher {
+    fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
+        f.debug_struct("BandwidthVoucher")
+            .field("serial_number", &self.serial_number)
+            .field("binding_number", &self.binding_number)
+            .field("voucher_value", &self.voucher_value)
+            .field("voucher_value_plain", &self.voucher_value_plain)
+            .field("voucher_info", &self.voucher_info)
+            .field("voucher_info_plain", &self.voucher_info_plain)
+            .field("tx_hash", &self.tx_hash)
+            .field("signing_key", &self.signing_key.to_base58_string())
+            .field("encryption_key", &self.encryption_key.to_base58_string())
+            .field(
+                "pedersen_commitments_openings",
+                &self.pedersen_commitments_openings,
+            )
+            .field("blind_sign_request", &self.blind_sign_request)
+            .finish()
+    }
+}
+
 impl BandwidthVoucher {
     pub fn new(
         params: &Parameters,
@@ -114,7 +135,7 @@ impl BandwidthVoucher {
     pub fn try_from_bytes(bytes: &[u8]) -> Result<Self, Error> {
         if bytes.len() < 32 * 5 + 4 * 8 {
             return Err(Error::BandwidthVoucherDeserializationError(format!(
-                "Less then {} bytes needed",
+                "Less than {} bytes needed",
                 32 * 5 + 4 * 8
             )));
         }

common/network-defaults/Cargo.toml

@@ -16,3 +16,11 @@ schemars = { version = "0.8", features = ["preserve_order"] }
 serde = { workspace = true, features = ["derive"]}
 thiserror = { workspace = true }
 url = { workspace = true }
+
+# 'wasm-serde-types' feature
+tsify = { workspace = true, features = ["js"], optional = true }
+wasm-bindgen = { workspace = true, optional = true }
+
+[features]
+default = []
+wasm-serde-types = ["tsify", "wasm-bindgen"]