Fix Jade signing issues with Swan Vault #2421

moneymanolis · 2024-03-13T17:59:18Z

Fixes #2402

This PR also includes:

Revert to call enumerate on start-up by default to fix HWI issues on master, changed enforcehwiinitialisation to skiphwiinitialisation
Update Jade API to version 2.0.2 and update the Jade hwi client to the latest hwi version
Avoiding the prompt of the Jade to unlock the device upon Specter start-up
Providing a UI for registering the multisig (currently only Jade) on the wallets --> setting page if a Jade is part of the multisig, see screenshot:

Tested

Jade
Trezor One
Trezor Model T

…test hwi code (version

… to skip_hwi_initialisation + some simplifications

…hanges to be on the safe side (avoid name collusion with built-in fetch + random id)

netlify · 2024-03-13T17:59:33Z

✅ Deploy Preview for specter-desktop-docs canceled.

Name	Link
🔨 Latest commit	`473b202`
🔍 Latest deploy log	https://app.netlify.com/sites/specter-desktop-docs/deploys/65f336a52bbaed00081925c1

moneymanolis · 2024-03-13T18:02:54Z

@JamieDriver could you check those Jade related commits:
add "unlock" to jade's enumerate method and its hwi client
add ui to register multisigs for multisig wallets using a jade

requirements.txt

src/cryptoadvance/specter/devices/hwi/jade.py

… return up in jade client

JamieDriver · 2024-03-15T12:38:55Z

All good with me.

moneymanolis added 7 commits March 8, 2024 11:45

update jade api to version 2.0.2 and update jade hwi client to the la…

3d404ec

…test hwi code (version

change chain default for initialising the jade client back to MAIN

11ce7bd

add "unlock" to jade's enumerate method and its hwi client

375159d

use is_startup property to prevent jade unlocking on startup + change…

30cb33c

… to skip_hwi_initialisation + some simplifications

pass chain param on every request for enumerate call as well + some c…

af429f0

…hanges to be on the safe side (avoid name collusion with built-in fetch + random id)

always use timeout when calling enumerate from hwi.jinja

42b8d61

add ui to register multisigs for multisig wallets using a jade

619f282

moneymanolis added bug Something isn't working enhancement New feature or request labels Mar 13, 2024

moneymanolis requested a review from stepansnigirev March 13, 2024 17:59

moneymanolis requested a review from k9ert March 13, 2024 18:03

updated requirements.txt

5046156