--- Day changed Mon May 06 2019
00:59 -!- fanquake [~fanquake@unaffiliated/fanquake] has quit []     
01:40 -!- fanquake [~fanquake@unaffiliated/fanquake] has joined ##hwi     
06:40 < instagibbs> achow101, have you gotten coldcard signing from sd working?
06:40 < instagibbs> i followed their steps, device doesn't see the file or something, never presented with a file list
07:17 < achow101> instagibbs: haven't tried it yet
07:52 -!- eldi [~eldi@2a01:e34:ed3e:7c10:c4d2:6148:9aff:e98c] has joined ##hwi     
07:52 < eldi> hi all ... i seem to have found a problem ...
07:54 < eldi> importmulti in bitcoin core 0.18 does not like the origin identifier info in the json desc data
08:01 < harding> Note, I don't know what the exact issue is, but I can confirm that eldi has what looks like a valid multisig descriptor that works with deriveaddresses and importmulti when the origin information is omited but does not work when the origin information is provided.  I don't see any obvious problems.  Since he's ultimately trying to get HWI to work, I suggested he ask here.
08:04 < eldi> I expected  HWI to provide Bitcoin Core compatible descriptors ... yet it seems it does not by default - can some switches be added to make it so ?
08:23 < eldi> a bit more info ... i was trying to create and importmulti a 2 of 3 multisig descriptor i had crafted manually .. using descriptors i had created with HWI .. i guess changes need to be made to bitcoin core with regard to the import multi allowing the inclusion of the origin info .. just wanted to make you aware of this issue
08:26 < harding> eldi: the origin info should be supported, and it worked when I tested importing each key individually.  It was only in the multisig configuration that it didn't work.  If nobody else picks up the problem, I'll look into it more later.
08:31 < eldi> and is it also worth noting that bitcoin-cli getdescriptorinfo reports using a ' in the path which is incompatible with importmulti which requires h to be used
08:36 < harding> eldi: you can probably use the "'", you just need to escape it properly (which is a pain on the CLI in general).
08:37 < harding> eldi: it may be worth testing whether that's your problem.
08:37 < harding> (Although the docs do say that h should be equivilent to ')
08:38 < eldi> ill give it a go thanks
09:26 < gwillen> there is a problem with h versus ' that can trip you up
09:26 < gwillen> the checksum is sensitive to which one you use, but if you use getdescrptorinfo to get a checksum, it always gives the ' version
09:26 < achow101> eldi: harding: what is the descriptor?
09:26 < gwillen> I complained and got people to agree in principle to change the default to h, so we can avoid escaping issues, but I don't know if there's a bug open for that
09:29 < achow101> eldi: HWI creates the complete importmulti command. However creating a multisig from the descriptors provided from that command is not necessarily straightforward.
09:32 < harding> gwillen: oh, that does sound like that could be the issue here.
09:34 < harding> eldi: I just tried with the ' form of the origin information and it works with deriveaddresses (so it'll probably work with importmulti if you get the escaping right).
09:36 < harding> gwillen: thanks!  Looks like this is you issue; I'll leave a comment on it supporting the switch from ' to h by default.  https://github.com/bitcoin/bitcoin/issues/15740
09:41 < eldi> an example of the descriptor that was giving me problems is
09:41 < eldi> "wsh(multi(2,[6079769c/84h/0h/0h]xpub6DLcWRLzQnCMwkz4k5MTkBe9ywCvCUdSown1ifPMVjwUwVUrhS8WPrAdpZcsHyoN4nkgKzvjLiHjHQnyHSk96NYxeUfUVytK2NjLsxr1jiW/0/*,[33021e6b/84h/0h/0h]xpub6DQh943hDhtw1Sk1MsZ1GQHeqqBbG1ya7P59vYpc2qrXJNsFtbU6geuQgRsnbv7vP5VPKZeDHhBFB5Bp7HEuMP4M5S4wns6cewBsdfxwcCz/0/*,[4c31ed24/84h/0h/0h]xpub6CbDwWWzktL4BVpjDyxzG4Za7MnikdhYu9vKY7XF7szA4BMpiCio8WizgEdzYNPnTvYiRwohQrrYAFH9643ddRAQoawCT3Su8weTFZoXYxp/0/*))#jph6un4q"
09:42 < eldi> as you can see i included the orgin into and should not have when importmulti .. it works with getdescriptorinfo but not derviveaddresses or importmulti due to the origin info
09:44 < harding> eldi: it does work if you switch each of the h to '.
09:46 < gwillen> harding: cool, gladd I could help
09:46 < gwillen> I ran into this when trying to test out descriptors in the console by hand
09:48 < eldi> i am not sure how to get the escaping correct
09:51 < eldi> i tried putting double quotes around the single quotes but thats not quite right
09:53 < harding> eldi: https://0bin.net/paste/ol5VcIv7CCH488ZU#o9QGLRRhLoPJeUBzsrT+KeDQj+PTn06Oy4sfeOsRk+m
09:53 < harding> eldi: note, that's not the exact command you want.
09:54 < harding> I had to change the keypool command to false on mine since I didn't have an empty wallet.
09:55 < eldi> i see how the string has changed and the escaping is different - thank you for that
09:57 < gwillen> the other trick you can use: to escape a ' inside 's, replace each ' with '\''
09:57 < gwillen> but the trick harding used works too
10:01 < eldi> ok i understand - but it seems ' is not replaceable with h as it appears in the docs
10:01 < eldi> i got the importmulti to work with ' and the escaping that harding demonstrated in the pastebin
10:06 < harding> eldi: it is replacable with h but, as gwillen mentioned, getdescriptorinfo is quietly changing h back to ' when you run it.
10:06 < harding> The checksum that getdescriptorinfo prints is thus only valid with the ' variation, not the h variation.
10:08 < eldi> ok ... i understand as the checksum is required on the importmulti
10:10 < gwillen> eldi: the problem is that getdescriptorinfo ONLY gives a checksum that works with ', never with h
10:10 < gwillen> (I filed a bug to change that, but it's not changed yet)
10:10 < gwillen> even if you use h, it changes them to '
10:11 < eldi> yes i noticed that
10:11 < eldi> would be nice if it returned it as provided
10:11 < eldi> with checksum
10:11 < gwillen> yeah, agree
10:11 < gwillen> actually I'd be much obliged if you could comment on https://github.com/bitcoin/bitcoin/issues/15740, and harding too, in support of the change
10:12 < gwillen> I am betwewen PRs just now and will go make it myself :-)
10:16 < eldi> done!
10:20 < achow101> technically HWI is more correct for using h and not '. bip32 actually specifies that hardened derivation should be specified using a subscript of H
10:30 < eldi> thanks to all who created HWI ... its very nice to be able to link HWW with Core
10:34 < achow101> I've written a script that computes the checksum for a descriptor: https://github.com/achow101/output-descriptors. You can use it to compute the checksum for descriptors that use h so you don't have to deal with escaping '
10:41 < eldi> thats neat - thanks for the link
10:57 < harding> gwillen: left a comment.  Thanks again!
11:54 < gwillen> absolutely, thanks for the comment, and thanks for the suggestion of including both options, I didn't think of that for some reason
12:39 -!- eldi [~eldi@2a01:e34:ed3e:7c10:c4d2:6148:9aff:e98c] has quit [Ping timeout: 248 seconds]     
12:41 -!- eldi [~eldi@2a01:e34:ed3e:7c10:55f8:3e7:b695:d471] has joined ##hwi     
17:02 -!- fanquake [~fanquake@unaffiliated/fanquake] has quit []     
23:10 -!- fanquake [~fanquake@unaffiliated/fanquake] has joined ##hwi