< kallewoof> jeremyrubin: it bugs out occasionally. and it also sometimes bans your account.
< kallewoof> updated btcdeb to latest taproot. still need to update the 'tap' command, but making taproot/tapscript spending seems to work (tried against regtest and testmepoolaccept works for both types, generated using btcdeb).
< kallewoof> this is a pull request only, so you'd need to grab that, btw.
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/a2a250c7d07b...c6532fa6c14d
< bitcoin-git> bitcoin/master fa41810 MarcoFalke: ci: Run valgrind fuzzer on cirrus
< bitcoin-git> bitcoin/master fa0538e MarcoFalke: ci: Set cirrus RAM to 8GB
< bitcoin-git> bitcoin/master c6532fa MarcoFalke: Merge #19746: ci: Move valgrind fuzzer to cirrus
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #19746: ci: Move valgrind fuzzer to cirrus (master...2008-ciValgrind) https://github.com/bitcoin/bitcoin/pull/19746
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #19765: doc: Fix getmempoolancestor RPC result doc (master...2008-rpcDoc) https://github.com/bitcoin/bitcoin/pull/19765
< hebasto> #19710 review begging
< gribble> https://github.com/bitcoin/bitcoin/issues/19710 | bench: Prevent thread oversubscription and decreases the variance of result values by hebasto · Pull Request #19710 · bitcoin/bitcoin · GitHub
< jonatack> review beg also for #19628 needed for BIP155 addrv2
< gribble> https://github.com/bitcoin/bitcoin/issues/19628 | net: change CNetAddr::ip to have flexible size by vasild · Pull Request #19628 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/c6532fa6c14d...44ddcd887ddf
< bitcoin-git> bitcoin/master f0fce06 Sebastian Falbesoner: util: make EncodeBase58 consume Spans
< bitcoin-git> bitcoin/master 356988e Sebastian Falbesoner: util: make EncodeBase58Check consume Spans
< bitcoin-git> bitcoin/master 44ddcd8 Wladimir J. van der Laan: Merge #19706: refactor: make EncodeBase58{Check} consume Spans
< bitcoin-git> [bitcoin] laanwj merged pull request #19706: refactor: make EncodeBase58{Check} consume Spans (master...20200810-util-make-base58encode-consume-spans) https://github.com/bitcoin/bitcoin/pull/19706
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/44ddcd887ddf...e9b30126545d
< bitcoin-git> bitcoin/master 71e0f07 Sebastian Falbesoner: util: remove unused c-string variant of atoi64()
< bitcoin-git> bitcoin/master e9b3012 Wladimir J. van der Laan: Merge #19750: refactor: remove unused c-string variant of atoi64()
< bitcoin-git> [bitcoin] laanwj merged pull request #19750: refactor: remove unused c-string variant of atoi64() (master...20200817-util-remove-unused_cstr_atoi64) https://github.com/bitcoin/bitcoin/pull/19750
< shesek> is there an easy way to tell whether a node is accepting incoming connections via the rpc?
< shesek> one way to do this is to look for peers with inbound:true in getpeerinfo, which I guess is the most definitive way to check that connections are accepted in practice
< jonatack> shesek: i proposed adding inbound and outbound connection counts to RPC getnetworkinfo in #19405
< gribble> https://github.com/bitcoin/bitcoin/issues/19405 | rpc, cli: add network in/out connections to `getnetworkinfo` and `-getinfo` by jonatack · Pull Request #19405 · bitcoin/bitcoin · GitHub
< jonatack> shesek: ATM #19643 is open to add an easy way to see them as well, review welcome
< gribble> https://github.com/bitcoin/bitcoin/issues/19643 | Add `-netinfo` peer connections dashboard by jonatack · Pull Request #19643 · bitcoin/bitcoin · GitHub
< shesek> but that's for the bitcoin-cli client and not something provided by the rpc itself, right?
< shesek> fwiw I would find #19405 pretty useful for my purposes
< gribble> https://github.com/bitcoin/bitcoin/issues/19405 | rpc, cli: add network in/out connections to `getnetworkinfo` and `-getinfo` by jonatack · Pull Request #19405 · bitcoin/bitcoin · GitHub
< jonatack> so did i, but it died for lack of review and then the review was clobbered by another PR
< jonatack> could maybe re-open with just the RPC portion, the changeset is really minimal
< jonatack> will do tomorrow am and ping you
< jonatack> meanwhile, there's bitcoin-cli getpeerinfo | jq "group_by(.inbound) | map({Inbound: .[0].inbound, Count: length}) | .[]"
< jonatack> fwiw it's merged into bitcoin-knots
< bitcoin-git> [bitcoin] jonatack reopened pull request #19405: rpc, cli: add network in/out connections to `getnetworkinfo` and `-getinfo` (master...in-and-out-connections) https://github.com/bitcoin/bitcoin/pull/19405
< jonatack> shesek: runless you set -listen=0, inbound connections should be accepted by default unless -proxy or -connect is used
< jonatack> e.g. bitcoind -h | grep -A1 "\-listen"
< shesek> jonatack, right, but I'm trying to determine that in the context of an application that connects to the node over rpc, I can't tell how the user's node is going to be configured
< jonatack> 👍
< shesek> I think this may be the first time I saw an emoji on irc
< sipa> Ừńìċộđẹ ổñ ỊŔÇ îś łờţš ọƒ ƒûň ŧħóǔğħ
< sipa> uıds pɐǝɥ ɹnoʎ sǝʞɐɯ ʎʃʃɐǝɹ ʇı
< luke-jr> -.-
< shesek> sipa, lol, it did :)
< sipa> ₿
< sipa> [A
< bitcoin-git> [bitcoin] luke-jr closed pull request #19243: banman: Limit resources consumed by misbehaving node deprioitisation (master...misbehaving_limit) https://github.com/bitcoin/bitcoin/pull/19243
< luke-jr> I would if it would improve review, to have a single reviewer responsible for the initial churn with the author, and only have others review after the two of them are happy with it