Added support for f4 addresses in forest-wallet

CHANGELOG.md

@@ -45,6 +45,8 @@ This is a non-mandatory release for all node operators. It enables F3 finality r
 
 ### Added
 
+- [#6710](https://github.com/ChainSafe/forest/pull/6710): Added support for f4 addresses in forest-wallet.
+
 ### Changed
 
 - [#6631](https://github.com/ChainSafe/forest/issues/6631): Backported F3 finality resolution to ETH v1 RPC methods.

scripts/tests/calibnet_wallet_check.sh

@@ -115,6 +115,8 @@ sleep 5s
 # Show balances
 $FOREST_WALLET_PATH list
 
+FOREST_URL="http://127.0.0.1:2345/rpc/v1"
+
 echo "Creating a new address to send FIL to"
 ADDR_TWO=$($FOREST_WALLET_PATH new)
 echo "$ADDR_TWO"
@@ -154,6 +156,113 @@ while [[ $i != 20 && $ADDR_THREE_BALANCE == "$FIL_ZERO" ]]; do
   ADDR_THREE_BALANCE=$($FOREST_WALLET_PATH --remote-wallet balance "$ADDR_THREE" --exact-balance)
 done
 
+ETH_ADDR_TWO=$(curl -s -X POST "$FOREST_URL" \
+  -H 'Content-Type: application/json' \
+  -H "Authorization: Bearer $ADMIN_TOKEN" \
+  --data "$(jq -n --arg addr "$ADDR_TWO" '{jsonrpc: "2.0", id: 1, method: "Filecoin.FilecoinAddressToEthAddress", params: [$addr, "pending"]}')" \
+  | jq -r '.result')
+echo "ETH address: $ETH_ADDR_TWO"
+
+ETH_ADDR_THREE=$(curl -s -X POST "$FOREST_URL" \
+  -H 'Content-Type: application/json' \
+  -H "Authorization: Bearer $ADMIN_TOKEN" \
+  --data "$(jq -n --arg addr "$ADDR_THREE" '{jsonrpc: "2.0", id: 1, method: "Filecoin.FilecoinAddressToEthAddress", params: [$addr, "pending"]}')" \
+  | jq -r '.result')
+echo "ETH address: $ETH_ADDR_THREE"
+
+MSG_ETH=$($FOREST_WALLET_PATH send "$ETH_ADDR_TWO" "$FIL_AMT")
+: "$MSG_ETH"
+
+MSG_ETH_REMOTE=$($FOREST_WALLET_PATH --remote-wallet send "$ETH_ADDR_THREE" "$FIL_AMT")
+: "$MSG_ETH_REMOTE"
+
+ETH_ADDR_TWO_BALANCE=$ADDR_TWO_BALANCE
+i=0
+while [[ $i != 20 && $ETH_ADDR_TWO_BALANCE == "$ADDR_TWO_BALANCE" ]]; do
+  i=$((i+1))
+
+  : "Checking balance $i/20"
+  sleep 30s
+  ETH_ADDR_TWO_BALANCE=$($FOREST_WALLET_PATH balance "$ADDR_TWO" --exact-balance)
+done
+
+ETH_ADDR_THREE_BALANCE=$ADDR_THREE_BALANCE
+i=0
+while [[ $i != 20 && $ETH_ADDR_THREE_BALANCE == "$ADDR_THREE_BALANCE" ]]; do
+  i=$((i+1))
+
+  : "Checking balance $i/20"
+  sleep 30s
+  ETH_ADDR_THREE_BALANCE=$($FOREST_WALLET_PATH --remote-wallet balance "$ADDR_THREE" --exact-balance)
+done
+
+: Begin delegated wallet tests
+
+# The following steps do basic delegated wallet handling tests.
+
+echo "Creating delegated wallet DELEGATE_ADDR_ONE"
+DELEGATE_ADDR_ONE=$($FOREST_WALLET_PATH new delegated)
+echo "$DELEGATE_ADDR_ONE"
+$FOREST_WALLET_PATH export "$DELEGATE_ADDR_ONE" > delegated_wallet.key
+$FOREST_WALLET_PATH --remote-wallet import delegated_wallet.key
+
+# Fund delegated wallet from preloaded wallet
+DELEGATE_FUND_AMT="2 micro FIL"
+$FOREST_WALLET_PATH set-default "$ADDR_ONE"
+MSG_DELEGATE_FUND=$($FOREST_WALLET_PATH send "$DELEGATE_ADDR_ONE" "$DELEGATE_FUND_AMT")
+: "$MSG_DELEGATE_FUND"
+
+DELEGATE_ADDR_ONE_BALANCE=$FIL_ZERO
+i=0
+while [[ $i != 20 && $DELEGATE_ADDR_ONE_BALANCE == "$FIL_ZERO" ]]; do
+  i=$((i+1))
+  : "Checking DELEGATE_ADDR_ONE balance $i/20"
+  sleep 30s
+  DELEGATE_ADDR_ONE_BALANCE=$($FOREST_WALLET_PATH balance "$DELEGATE_ADDR_ONE" --exact-balance)
+done
+
+echo "Creating delegated wallet DELEGATE_ADDR_TWO"
+DELEGATE_ADDR_TWO=$($FOREST_WALLET_PATH new delegated)
+echo "$DELEGATE_ADDR_TWO"
+$FOREST_WALLET_PATH set-default "$DELEGATE_ADDR_ONE"
+
+echo "Creating delegated (remote) wallet DELEGATE_ADDR_THREE"
+DELEGATE_ADDR_THREE=$($FOREST_WALLET_PATH --remote-wallet new delegated)
+echo "$DELEGATE_ADDR_THREE"
+$FOREST_WALLET_PATH --remote-wallet set-default "$DELEGATE_ADDR_ONE"
+
+$FOREST_WALLET_PATH list
+$FOREST_WALLET_PATH --remote-wallet list
+
+MSG_DELEGATE_TWO=$($FOREST_WALLET_PATH send "$DELEGATE_ADDR_TWO" "$FIL_AMT")
+: "$MSG_DELEGATE_TWO"
+
+MSG_DELEGATE_THREE=$($FOREST_WALLET_PATH send "$DELEGATE_ADDR_THREE" "$FIL_AMT")
+: "$MSG_DELEGATE_THREE"
+
+DELEGATE_ADDR_TWO_BALANCE=$FIL_ZERO
+i=0
+while [[ $i != 20 && $DELEGATE_ADDR_TWO_BALANCE == "$FIL_ZERO" ]]; do
+  i=$((i+1))
+  : "Checking DELEGATE_ADDR_TWO balance $i/20"
+  sleep 30s
+  DELEGATE_ADDR_TWO_BALANCE=$($FOREST_WALLET_PATH balance "$DELEGATE_ADDR_TWO" --exact-balance)
+done
+
+DELEGATE_ADDR_THREE_BALANCE=$FIL_ZERO
+i=0
+while [[ $i != 20 && $DELEGATE_ADDR_THREE_BALANCE == "$FIL_ZERO" ]]; do
+  i=$((i+1))
+  : "Checking DELEGATE_ADDR_THREE balance $i/20"
+  sleep 30s
+  DELEGATE_ADDR_THREE_BALANCE=$($FOREST_WALLET_PATH --remote-wallet balance "$DELEGATE_ADDR_THREE" --exact-balance)
+done
+
+$FOREST_WALLET_PATH list
+$FOREST_WALLET_PATH --remote-wallet list
+
+: End delegated wallet tests
+
 # wallet list should contain address two with transferred FIL amount
 $FOREST_WALLET_PATH list
 $FOREST_WALLET_PATH --remote-wallet list

src/message_pool/msgpool/msg_pool.rs

@@ -16,8 +16,9 @@ use crate::eth::is_valid_eth_tx_for_sending;
 use crate::libp2p::{NetworkMessage, PUBSUB_MSG_STR, Topic};
 use crate::message::{ChainMessage, Message, SignedMessage, valid_for_block_inclusion};
 use crate::networks::{ChainConfig, NEWEST_NETWORK_VERSION};
+use crate::rpc::eth::types::EthAddress;
 use crate::shim::{
-    address::Address,
+    address::{Address, Protocol},
     crypto::{Signature, SignatureType},
     econ::TokenAmount,
     gas::{Gas, price_list_by_network_version},
@@ -265,6 +266,12 @@ where
         if to_vec(msg)?.len() > MAX_MESSAGE_SIZE {
             return Err(Error::MessageTooBig);
         }
+        let to = msg.message().to();
+        if to.protocol() == Protocol::Delegated && EthAddress::from_filecoin_address(&to).is_err() {
+            return Err(Error::Other(format!(
+                "message recipient {to} is a delegated address but not a valid Eth Address"
+            )));
+        }
         valid_for_block_inclusion(msg.message(), Gas::new(0), NEWEST_NETWORK_VERSION)?;
         if msg.value() > *crate::shim::econ::TOTAL_FILECOIN {
             return Err(Error::MessageValueTooHigh);

src/rpc/methods/eth.rs

@@ -1012,7 +1012,7 @@ async fn execute_tipset<DB: Blockstore + Send + Sync + 'static>(
     ))
 }
 
-fn is_eth_address(addr: &VmAddress) -> bool {
+pub fn is_eth_address(addr: &VmAddress) -> bool {
     if addr.protocol() != Protocol::Delegated {
         return false;
     }