Limit key name length

stellar · Dec 23, 2024 · 95d8b32 · 95d8b32
1 parent e95fa3a
commit 95d8b32
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/cmd/soroban-cli/src/config/address.rs b/cmd/soroban-cli/src/config/address.rs
@@ -28,7 +28,11 @@ pub enum Error {
     Secret(#[from] secret::Error),
     #[error("Address cannot be used to sign {0}")]
     CannotSign(xdr::MuxedAccount),
-    #[error("Invalid key name: {0}\n only alphanumeric characters, `_`and `-` are allowed")]
+    #[error("Invalid key name: {0}\n only alphanumeric characters, underscores (_), and hyphens (-) are allowed.")]
+    InvalidKeyNameCharacters(String),
+    #[error("Invalid key name: {0}\n keys cannot exceed 250 characters")]
+    InvalidKeyNameLength(String),
+    #[error("Invalid key name: {0}\n keys cannot be the word \"ledger\"")]
     InvalidKeyName(String),
 }
 
@@ -93,11 +97,14 @@ impl std::str::FromStr for KeyName {
     type Err = Error;
     fn from_str(s: &str) -> Result<Self, Self::Err> {
         if !s.chars().all(allowed_char) {
-            return Err(Error::InvalidKeyName(s.to_string()));
+            return Err(Error::InvalidKeyNameCharacters(s.to_string()));
         }
         if s == "ledger" {
             return Err(Error::InvalidKeyName(s.to_string()));
         }
+        if s.len() > 250 {
+            return Err(Error::InvalidKeyNameLength(s.to_string()));
+        }
         Ok(KeyName(s.to_string()))
     }
 }