Return error when trying to get Secure Store or Ledger secret

stellar · Dec 2, 2024 · a6756f1 · a6756f1
1 parent 9391731
commit a6756f1
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/cmd/soroban-cli/src/config/secret.rs b/cmd/soroban-cli/src/config/secret.rs
@@ -32,6 +32,8 @@ pub enum Error {
     LedgerDoesNotRevealSecretKey,
     #[error(transparent)]
     Keyring(#[from] keyring::Error),
+    #[error("Secure Store does not reveal secret key")]
+    SecureStoreDoesNotRevealSecretKey,
 }
 
 #[derive(Debug, clap::Args, Clone)]
@@ -129,8 +131,12 @@ impl Secret {
                     .private()
                     .0,
             )?,
-            Secret::Ledger => panic!("Ledger does not reveal secret key"),
-            Secret::SecureStore { .. } => panic!("Secure Store does not reveal secret key"),
+            Secret::Ledger => {
+                return Err(Error::LedgerDoesNotRevealSecretKey);
+            }
+            Secret::SecureStore { .. } => {
+                return Err(Error::SecureStoreDoesNotRevealSecretKey);
+            }
         })
     }