fixup! eth: add signing function parameter to specify receive address case

Author: Tomasvrba

CHANGELOG-npm.md

@@ -6,6 +6,7 @@
 - btc: handle error when an input's previous transaction is required but missing
 - btc: add support for regtest
 - btc: add support for Taproot wallet policies
+- eth: add method to help clients identify address case (upper/lower/mixed)
 
 ## 0.6.0
 

CHANGELOG-rust.md

@@ -7,6 +7,7 @@
 - btc: add support for regtest
 - btc: add support for Taproot wallet policies
 - cardano: added support for vote delegation
+- eth: add method to help clients identify address case (upper/lower/mixed)
 
 ## 0.5.0
 

examples/eth.rs

@@ -81,6 +81,7 @@ async fn eth_demo<R: bitbox_api::runtime::Runtime>() {
             1,
             &"m/44'/60'/0'/0/0".try_into().unwrap(),
             &raw_tx.as_slice().try_into().unwrap(),
+            None,
         )
         .await
         .unwrap();
@@ -92,6 +93,7 @@ async fn eth_demo<R: bitbox_api::runtime::Runtime>() {
         .eth_sign_1559_transaction(
             &"m/44'/60'/0'/0/0".try_into().unwrap(),
             &raw_tx.as_slice().try_into().unwrap(),
+            None,
         )
         .await
         .unwrap();

src/eth.rs

@@ -123,12 +123,18 @@ pub struct EIP1559Transaction {
 }
 
 /// Identifies the case of the recipient address given as hexadecimal string.
-/// This function exists as a convenience to potentially help clients to determine the case of the
+/// This function exists as a convenience to help clients to determine the case of the
 /// recipient address.
 pub fn eth_identify_case(recipient_address: &str) -> pb::EthAddressCase {
-    if recipient_address.to_uppercase() == recipient_address {
+    if recipient_address
+        .chars()
+        .all(|c| !c.is_ascii_alphabetic() || c.is_ascii_uppercase())
+    {
         pb::EthAddressCase::Upper
-    } else if recipient_address.to_lowercase() == recipient_address {
+    } else if recipient_address
+        .chars()
+        .all(|c| !c.is_ascii_alphabetic() || c.is_ascii_lowercase())
+    {
         pb::EthAddressCase::Lower
     } else {
         pb::EthAddressCase::Mixed
@@ -1267,4 +1273,20 @@ mod tests {
         )
         .is_err());
     }
+
+    #[test]
+    fn test_eth_identify_case() {
+        assert_eq!(
+            eth_identify_case("0XF39FD6E51AAD88F6F4CE6AB8827279CFFFB92266"),
+            pb::EthAddressCase::Upper
+        );
+        assert_eq!(
+            eth_identify_case("0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266"),
+            pb::EthAddressCase::Lower
+        );
+        assert_eq!(
+            eth_identify_case("0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266"),
+            pb::EthAddressCase::Mixed
+        );
+    }
 }

src/wasm/mod.rs

@@ -419,7 +419,12 @@ impl PairedBitBox {
     ) -> Result<types::TsEthSignature, JavascriptError> {
         let signature = self
             .device
-            .eth_sign_transaction(chain_id, &keypath.try_into()?, &tx.try_into()?, address_case.map(TryInto::try_into).transpose()?)
+            .eth_sign_transaction(
+                chain_id,
+                &keypath.try_into()?,
+                &tx.try_into()?,
+                address_case.map(TryInto::try_into).transpose()?,
+            )
             .await?;
 
         let v: u64 = compute_v(chain_id, signature[64])
@@ -443,7 +448,11 @@ impl PairedBitBox {
     ) -> Result<types::TsEthSignature, JavascriptError> {
         let signature = self
             .device
-            .eth_sign_1559_transaction(&keypath.try_into()?, &tx.try_into()?, address_case.map(TryInto::try_into).transpose()?)
+            .eth_sign_1559_transaction(
+                &keypath.try_into()?,
+                &tx.try_into()?,
+                address_case.map(TryInto::try_into).transpose()?,
+            )
             .await?;
 
         Ok(serde_wasm_bindgen::to_value(&types::EthSignature {

src/wasm/types.rs

@@ -75,7 +75,7 @@ type EthSignature = {
   s: Uint8Array;
   v: Uint8Array;
 };
-type EthAddressCase = 'Upper' | 'Lower' | 'Mixed';
+type EthAddressCase = 'upper' | 'lower' | 'mixed';
 type CardanoXpub = Uint8Array;
 type CardanoXpubs = CardanoXpub[];
 type CardanoNetwork = 'mainnet' | 'testnet';
@@ -287,17 +287,17 @@ impl TryFrom<TsEth1559Transaction> for crate::eth::EIP1559Transaction {
 }
 
 impl TryFrom<TsEthAddressCase> for crate::pb::EthAddressCase {
-  type Error = JavascriptError;
-  fn try_from(value: TsEthAddressCase) -> Result<Self, Self::Error> {
-      serde_wasm_bindgen::from_value(value.into())
-          .map_err(|_| JavascriptError::InvalidType("wrong type for EthAddressCase"))
-  }
+    type Error = JavascriptError;
+    fn try_from(value: TsEthAddressCase) -> Result<Self, Self::Error> {
+        serde_wasm_bindgen::from_value(value.into())
+            .map_err(|_| JavascriptError::InvalidType("wrong type for EthAddressCase"))
+    }
 }
 
 impl From<crate::pb::EthAddressCase> for TsEthAddressCase {
-  fn from(case: crate::pb::EthAddressCase) -> Self {
-      serde_wasm_bindgen::to_value(&case).unwrap().into()
-  }
+    fn from(case: crate::pb::EthAddressCase) -> Self {
+        serde_wasm_bindgen::to_value(&case).unwrap().into()
+    }
 }
 
 impl TryFrom<TsCardanoNetwork> for crate::pb::CardanoNetwork {