Commit f91e58f
committed
fix(sdk-coin-flrp): custody wallets no longer require a password
FLRP (Flare P-Chain) is a TSS-only coin — its statics define CoinFeature.TSS
and CoinFeature.MPCV2 but not CoinFeature.MULTISIG. Custody wallets for FLRP
must therefore be created with multisigType='tss', and signing is handled
server-side with no user key material on the client.
Previously, getDefaultMultisigType() returned 'onchain'. This caused
generateWallet({ type: 'custodial' }) to take the onchain-custodial code path
in wallets.ts (line 548), which creates a wallet object whose multisigType the
platform rejects ("Source FLR P wallet is not MPC (multisigType: onchain)").
Any caller that did pass multisigType:'tss' explicitly would correctly use
generateCustodialMpcWallet (no passphrase), but without a matching default the
omission silently produced a broken wallet.
Fix: return multisigTypes.tss from getDefaultMultisigType(), matching the
behaviour of sdk-coin-flr (FLR C-Chain), which correctly defaults to TSS and
has working custody wallets. With this default, generateWallet({ type:
'custodial' }) for FLRP now routes through generateCustodialMpcWallet, which
creates the wallet server-side and requires no passphrase.
Ticket: CECHO-1451
Session-Id: 7a278a4a-2a01-4004-b7bf-1ec86294141e
Task-Id: f3d71be2-ccd9-4342-a175-d342d5857ebe1 parent d426bd5 commit f91e58f
3 files changed
Lines changed: 40 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
810 | 810 | | |
811 | 811 | | |
812 | 812 | | |
| 813 | + | |
| 814 | + | |
| 815 | + | |
| 816 | + | |
| 817 | + | |
| 818 | + | |
| 819 | + | |
| 820 | + | |
| 821 | + | |
| 822 | + | |
| 823 | + | |
| 824 | + | |
| 825 | + | |
| 826 | + | |
| 827 | + | |
| 828 | + | |
| 829 | + | |
| 830 | + | |
| 831 | + | |
| 832 | + | |
| 833 | + | |
| 834 | + | |
| 835 | + | |
| 836 | + | |
| 837 | + | |
| 838 | + | |
| 839 | + | |
| 840 | + | |
| 841 | + | |
| 842 | + | |
| 843 | + | |
| 844 | + | |
| 845 | + | |
| 846 | + | |
| 847 | + | |
| 848 | + | |
| 849 | + | |
| 850 | + | |
813 | 851 | | |
814 | 852 | | |
815 | 853 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
71 | | - | |
| 71 | + | |
72 | 72 | | |
73 | 73 | | |
74 | 74 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
67 | | - | |
| 67 | + | |
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
| |||
0 commit comments