chore(cli): validate hex key provided to wallet create --key

sn_cli/src/bin/subcommands/wallet/hot_wallet.rs

@@ -13,6 +13,7 @@ use super::{
 };
 use crate::get_stdin_response;
 
+use autonomi::utils::is_valid_key_hex;
 use bls::SecretKey;
 use clap::Parser;
 use color_eyre::{
@@ -176,6 +177,13 @@ pub(crate) async fn wallet_cmds_without_client(cmds: &WalletCmds, root_dir: &Pat
                     "Only one of `--key` or `--derivation` may be specified"
                 ));
             }
+            if let Some(key) = key {
+                // Check if key is valid
+                // Doing this early to avoid stashing an existing wallet while the provided key is invalid
+                if !is_valid_key_hex(key) {
+                    return Err(eyre!("Please provide a valid secret key in hex format. It must be 64 characters long."));
+                }
+            }
             // Check for existing wallet
             if let Ok(existing_wallet) = WalletApiHelper::load_from(root_dir) {
                 let balance = existing_wallet.balance();

sn_cli/src/lib.rs

@@ -8,6 +8,7 @@
 
 mod acc_packet;
 mod files;
+pub mod utils;
 
 pub use acc_packet::AccountPacket;
 pub use files::{

sn_cli/src/utils.rs

@@ -0,0 +1,4 @@
+/// Returns whether a hex string is a valid secret key in hex format.
+pub fn is_valid_key_hex(hex: &str) -> bool {
+    hex.len() == 64 && hex.chars().all(|c| c.is_ascii_hexdigit())
+}