< sipa>
jonasschnelli: will have a look tomorrow +and remind me if i forget)
< jonasschnelli>
sipa: sure. Thanks. No hurry and sorry for the bother
< * wumpus>
wishes we had called 'signrawtransactionwithwallet' just 'signwithwallet'
< * gmaxwell>
wishes someone had suggested that
< wumpus>
yes, I don't mean it as criticism, just reflection after some use
< wumpus>
it doesn't matter for programmatic use; I guess aliases on bitcoin-cli could be a thing
< gmaxwell>
bash aliases are a thing too.. if we break the interface again we could remember this and rename it.
< wumpus>
I hope that won't be necessary any time soon! anyhow, might add something like 'keep RPC command names short, put documentation and further descrption in the help; rationale: short names tend to be easier to remember and save typing' to the developer notes
< gmaxwell>
+1 I think get got signrawtransactionwithwallet because already a bunch of rpcs are really long
< Sentineo>
hm maybe if the rpc interface could narrow down the search when you input less chars than the full length, like 'sh ip ro' in cisco's CLI instead of show ip route
< gmaxwell>
makes adding any new command dangerous
< echeveria>
seems dangerous in a system handling money.
< gmaxwell>
you can use bash programmable completion so that tab works with bitcoin-cli if you really want...
< echeveria>
a helpful script grabs your input and slams out your BTC into the abyss.
< echeveria>
that's what you wanted, right, right?
< gmaxwell>
of course the bright bulbs w/ fedora first thing they did with programmable completion was use it to make your system make network requests (and wait for them to time out) for 'missing commands' that you tried to complete.
< gmaxwell>
presumably leaking god knows how many private keys to their servers...
< Sentineo>
did not think of that, lol. And yeah, true.
< fanquake>
jonatack Just fyi, you shouldn't have to manually add commits and authors to the 0.18 release notes, wumpus/someone will have a script to update them at the next rc.
< wumpus>
right, it makes coordination really difficult if everyone updates those lists
< wumpus>
(this will result in people and PRs added multiple times)
< wumpus>
shouldn't be necessary as they're (initially) generated using scripts
< wumpus>
gmaxwell: oh no; !!of course!! someone has to add network requests
< wumpus>
gmaxwell: I do know that ubuntu does it in case of completion of filesnames for scp (which is reasonably useful, but also potentially a leak), but contacting the servers of the distribution for list of commands seems a bad idea
< gmaxwell>
I think they changed it so that it doesn't leak the command at least anymore, but it does cause obvious delays still.. first thing I do on any fedora system is uninstall packagekit-command-not-found
< wumpus>
reminds me of GUI input fields leaking their contents for completion/spellcheck in real time, very good for passphrases
< gmaxwell>
there was some bitcoin wallet that was spellcheck leaking its key field recently
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #15831: test: Add test that addmultisigaddress fails for watchonly addresses (master...1904-testMulti) https://github.com/bitcoin/bitcoin/pull/15831