fix: add wallet type check for sign transaction

Ticket: WP-2664 TICKET: WP-2664
BitGo · Sep 16, 2024 · ddc04f5 · ddc04f5
1 parent a10dac8
commit ddc04f5
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/modules/sdk-core/src/bitgo/wallet/wallet.ts b/modules/sdk-core/src/bitgo/wallet/wallet.ts
@@ -1831,7 +1831,12 @@ export class Wallet implements IWallet {
       params.txPrebuild = { txRequestId };
     }
 
-    if (params.walletPassphrase && !(params.keychain || params.key)) {
+    if (
+      params.walletPassphrase &&
+      !(params.keychain || params.key) &&
+      (this.type() === 'hot' || this.type() === undefined)
+    ) {
+      // this logic should only apply to hot wallets
       if (!_.isString(params.walletPassphrase)) {
         throw new Error('walletPassphrase must be a string');
       }