--- Day changed Wed Jan 09 2019 05:08 -!- provoostenator changed the topic of ##hwi to: Hardware Wallet Interaction scripts | https://github.com/achow101/HWI | Core branch to use: https://github.com/achow101/bitcoin/tree/hww 07:05 < instagibbs> meshcollider, not sure if there's anything to be done here or if it was supposed to be obvious or whatever, but when you import a descriptor with any number of pubkeys, if the pubkeys already exist in the wallet their origin info gets over-written. So for a multisig they have to remember to hand it the key origin info for the key it can sign for 07:06 < instagibbs> otherwise it forgets entirely how the pubkey was generated 07:15 < instagibbs> also I haven't tried this using the "legacy" flow, but you cannot import both sh(wsh(S)) and wsh(S) which means you cannot have them in your address book using importmulti. Again, not sure if that was the intention but it complains you already know the privkey(???) 07:15 < instagibbs> Maybe I'm abusing importmulti but it's so much easier than figuring out the byzantine list of commands/args to import a solveable watchonly script otherwise :) 07:33 < instagibbs> https://github.com/achow101/HWI/pull/97 multisig spending test, currently only enabled for ledger since it's the only supported test one for now in master 10:04 <@provoostenator> instagibbs: does this multisig import automatically use the same derivation index for each key when you call getnewaddress? 10:05 < achow101> provoostenator: multisigs aren't imported to the keypool 10:05 < achow101> it wouldn't work 10:05 <@provoostenator> Or you testing only a signle address? 10:07 <@provoostenator> Ok, then I'm a bit confused about what the test in 97 is doing 10:07 < achow101> provoostenator: it's single address. you can't getnewaddress a multisig address without a lot more changes to the wallet. existing hww would not be able to do that 10:07 <@provoostenator> I guess that's something a true descriptor based wallet should be able to do, right? 10:08 < achow101> it's importing multisigs to the wallet and then figuring out their addresses so coins can be sent to them. instead of addmultisigaddress, it just uses importmulti 10:08 < achow101> right, a fully descriptor based wallet could getnewaddress a multisig address 11:34 -!- ghost43 [~daer@gateway/tor-sasl/daer] has quit [Ping timeout: 256 seconds] 11:35 -!- ghost43 [~daer@gateway/tor-sasl/daer] has joined ##hwi 13:54 < instagibbs> provoostenator, it tests multisig... 18:05 -!- achow101 [~achow101@unaffiliated/achow101] has quit [Ping timeout: 250 seconds] 18:16 -!- achow101 [~achow101@unaffiliated/achow101] has joined ##hwi