Remove panic for proper error

tari-project · May 15, 2024 · 2f7f267 · 2f7f267
1 parent e01e5f3
commit 2f7f267
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/base_layer/core/src/transactions/key_manager/inner.rs b/base_layer/core/src/transactions/key_manager/inner.rs
@@ -422,7 +422,7 @@ where TBackend: KeyManagerBackend<PublicKey> + 'static
                 Ok(key)
             },
             KeyId::Derived { branch, label, index } => match &self.wallet_type {
-                WalletType::Ledger(_) => panic!(),
+                WalletType::Ledger(_) => Err(KeyManagerServiceError::LedgerPrivateKeyInaccessible),
                 WalletType::Software(k, _pk) => {
                     let km = self
                         .key_managers

diff --git a/base_layer/key_manager/src/key_manager_service/error.rs b/base_layer/key_manager/src/key_manager_service/error.rs
@@ -51,6 +51,8 @@ pub enum KeyManagerServiceError {
     UnknownError(String),
     #[error("Ledger error: `{0}`")]
     LedgerError(String),
+    #[error("The Ledger private key cannot be access or read")]
+    LedgerPrivateKeyInaccessible,
 }
 
 impl From<RangeProofError> for KeyManagerServiceError {