Merge pull request #98 from BitGo/WP-5454-fix-eddsa-recovery-validation

pranavjain97 · web-flow · commit 188f19252d51 · 2025-07-30T15:04:24.000-04:00
fix(aw): fix non-sol eddsa recovery validation
diff --git a/src/api/master/handlers/recoveryWallet.ts b/src/api/master/handlers/recoveryWallet.ts
@@ -70,7 +70,7 @@ function validateRecoveryParams(
     return;
   }
 
-  if (isEddsaCoin(sdkCoin)) {
+  if (isEddsaCoin(sdkCoin) && sdkCoin.getFamily() === CoinFamily.SOL) {
     // EdDSA coins (like Solana) need solanaRecoveryOptions for standard recovery
     if (!params.solanaRecoveryOptions) {
       throw new ValidationError('Solana recovery options are required for EdDSA coin recovery');

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ function validateRecoveryParams(`
`70`	`70`	`return;`
`71`	`71`	`}`
`72`	`72`
`73`		`- if (isEddsaCoin(sdkCoin)) {`
	`73`	`+ if (isEddsaCoin(sdkCoin) && sdkCoin.getFamily() === CoinFamily.SOL) {`
`74`	`74`	`// EdDSA coins (like Solana) need solanaRecoveryOptions for standard recovery`
`75`	`75`	`if (!params.solanaRecoveryOptions) {`
`76`	`76`	`throw new ValidationError('Solana recovery options are required for EdDSA coin recovery');`