< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/92fee79dab38...7c8e605bf420
< bitcoin-git> bitcoin/master 9cc8e30 fanquake: test: fix sign comparison warning in socket tests
< bitcoin-git> bitcoin/master 7c8e605 fanquake: Merge #21159: test: fix sign comparison warning in socket tests
< bitcoin-git> [bitcoin] fanquake merged pull request #21159: test: fix sign comparison warning in socket tests (master...sign_compare_sock_tests) https://github.com/bitcoin/bitcoin/pull/21159
< bitcoin-git> [bitcoin] fanquake opened pull request #21205: build: actually fail when Boost is missing (master...actually_fail_when_boost_missing) https://github.com/bitcoin/bitcoin/pull/21205
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7c8e605bf420...36be9b821ae0
< bitcoin-git> bitcoin/master 901f543 Carl Dong: guix: Passthrough BASE_CACHE into container
< bitcoin-git> bitcoin/master 36be9b8 fanquake: Merge #21087: guix: Passthrough BASE_CACHE into container
< bitcoin-git> [bitcoin] fanquake merged pull request #21087: guix: Passthrough BASE_CACHE into container (master...2020-12-guix-base-cache) https://github.com/bitcoin/bitcoin/pull/21087
< bitcoin-git> [bitcoin] ryanofsky opened pull request #21206: refactor: Make CWalletTx sync state type-safe (master...pr/txstate) https://github.com/bitcoin/bitcoin/pull/21206
< bitcoin-git> [bitcoin] ryanofsky opened pull request #21207: MOVEONLY: CWallet transaction code out of wallet.cpp/.h (master...pr/txmove) https://github.com/bitcoin/bitcoin/pull/21207
< bitcoin-git> [bitcoin] fanquake opened pull request #21209: build: use newer source for libnatpmp (master...newer_libnatpmp) https://github.com/bitcoin/bitcoin/pull/21209
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #21210: doc: Rework internal and external links (master...2102-docLinks) https://github.com/bitcoin/bitcoin/pull/21210
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/36be9b821ae0...69f7f50aa5b7
< bitcoin-git> bitcoin/master de85af5 Sebastian Falbesoner: test: store subversion (user agent) as string in msg_version
< bitcoin-git> bitcoin/master 69f7f50 MarcoFalke: Merge #20993: test: store subversion (user agent) as string in msg_version...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #20993: test: store subversion (user agent) as string in msg_version (master...2021-01-test-store_user_agent_as_string) https://github.com/bitcoin/bitcoin/pull/20993
< vasild> I have noticed before that putting too low bandwidth setting on i2pd could cause some similar issues, or also there is an option to run in "hidden" mode or something like that
< vasild> it caused problems too
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/69f7f50aa5b7...7f831346cb73
< bitcoin-git> bitcoin/master fd0be92 practicalswift: doc: Add instructions on how to fuzz the P2P layer using Honggfuzz NetDriv...
< bitcoin-git> bitcoin/master 7f83134 MarcoFalke: Merge #20380: doc: Add instructions on how to fuzz the P2P layer using Hon...
< vasild> can you browse any i2p web sites through the socks5 proxy?
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #20380: doc: Add instructions on how to fuzz the P2P layer using Honggfuzz NetDriver (master...honggfuzz-p2p-fuzzing) https://github.com/bitcoin/bitcoin/pull/20380
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7f831346cb73...8639c446d88b
< bitcoin-git> bitcoin/master fafddfa MarcoFalke: scripted-diff: Remove shadowing lock annotations
< bitcoin-git> bitcoin/master 8639c44 MarcoFalke: Merge #21188: scripted-diff: Remove redundant lock annotations in net proc...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #21188: scripted-diff: Remove redundant lock annotations in net processing (master...2102-netProcNoShadow) https://github.com/bitcoin/bitcoin/pull/21188
< vasild> s/socks5/http/
< vasild> for example http://i2p-projekt.i2p/en/faq
< wumpus> vasild: it was clock issue apparently :-)
< wumpus> no, i couldn't do anything with it
< vasild> ooo
< wumpus> bitcoind and tor have a larger tolerance for clock drift than i2pd apparently
< vasild> clock issue + obscure error messages from i2pd, how did you figure it out?
< wumpus> i eventurally figured it out by looking carefully at the initial messages after i2pd's start instead of the spam afterward, the exact message is here: https://github.com/bitcoin/bitcoin/pull/20685#issuecomment-780412089 ... it's pretty clear in retrospect😳 (but might be something to document in the gotchas)
< wumpus> otherwise, setting it up was *so easy*
< bitcoin-git> [bitcoin] MarcoFalke pushed 6 commits to master: https://github.com/bitcoin/bitcoin/compare/8639c446d88b...569b5ba1dc56
< bitcoin-git> bitcoin/master 9a3bbe8 Amiti Uttarwar: [test] Introduce a unit test helper to create a valid mempool transaction.
< bitcoin-git> bitcoin/master a2d908e Amiti Uttarwar: [test] Throw error instead of segfaulting in failure scenario
< bitcoin-git> bitcoin/master df6a5fc Amiti Uttarwar: [util] Change GetMockTime to return chrono type instead of int
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #21121: [test] Small unit test improvements, including helper to make mempool transaction (master...2021-01-unit-test-valid-tx) https://github.com/bitcoin/bitcoin/pull/21121
< aj> MarcoFalke: do you know if it's possible to retrigger the timed out fuzzer CI tests so that they'll pick up #21185 and turn green?
< gribble> https://github.com/bitcoin/bitcoin/issues/21185 | fuzz: Remove expensive and redundant muhash from crypto fuzz target by MarcoFalke · Pull Request #21185 · bitcoin/bitcoin · GitHub
< aj> oops, should check my battery level before asking questions. bbl
< bitcoin-git> [bitcoin] martinus closed pull request #21176: Improve speed of Base58 Encoding (master...2021-02-optimize-base58) https://github.com/bitcoin/bitcoin/pull/21176
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #21211: test: Move P2WSH_OP_TRUE to shared test library (master...2102-testMoveScript) https://github.com/bitcoin/bitcoin/pull/21211
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/569b5ba1dc56...4dc1ff809a5d
< bitcoin-git> bitcoin/master 882ce25 Wladimir J. van der Laan: cli: Treat high detail levels as the maximum in -netinfo
< bitcoin-git> bitcoin/master 4dc1ff8 Wladimir J. van der Laan: Merge #21192: cli: Treat high detail levels as maximum in -netinfo
< bitcoin-git> [bitcoin] laanwj merged pull request #21192: cli: Treat high detail levels as maximum in -netinfo (master...2021-02-netinfo-verbosity) https://github.com/bitcoin/bitcoin/pull/21192
< bitcoin-git> [bitcoin] fanquake closed pull request #18605: build: Link time garbage collection (master...link_time_garbage_collection) https://github.com/bitcoin/bitcoin/pull/18605
< bitcoin-git> [bitcoin] ultramassiveblackhole opened pull request #21213: Legal requirement (master...patch-1) https://github.com/bitcoin/bitcoin/pull/21213
< bitcoin-git> [bitcoin] meshcollider closed pull request #21213: Legal requirement (master...patch-1) https://github.com/bitcoin/bitcoin/pull/21213
< bitcoin-git> [bitcoin] ultramassiveblackhole opened pull request #21214: You put back the name of Satoshi Nakamoto to the copyright notice of I fucking kill all of you! (master...patch-1) https://github.com/bitcoin/bitcoin/pull/21214
< bitcoin-git> [bitcoin] fanquake closed pull request #21214: You put back the name of Satoshi Nakamoto to the copyright notice of I fucking kill all of you! (master...patch-1) https://github.com/bitcoin/bitcoin/pull/21214
< fanquake> sipa / wumpus: please block ultramassiveblackhole
< MarcoFalke> aj, just click on "re-run"
< wumpus> fanquake done
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/4dc1ff809a5d...04336086d3cb
< bitcoin-git> bitcoin/master 7bf04e3 fanquake: build: remove mostly pointless BOOST_PROCESS macro
< bitcoin-git> bitcoin/master 0433608 Wladimir J. van der Laan: Merge #21182: build: remove mostly pointless BOOST_PROCESS macro
< bitcoin-git> [bitcoin] laanwj merged pull request #21182: build: remove mostly pointless BOOST_PROCESS macro (master...boost_process_nonsense) https://github.com/bitcoin/bitcoin/pull/21182
< Kiminuo> Hi all, I have seen this issue https://github.com/bitcoin/bitcoin/issues/21005 and I've come up with this https://github.com/kiminuo/bitcoin/commit/3323455e53751b41adf2629ba192d4d786894a71. What do you think about this approach?
< Kiminuo> I found a PR that does, in essence, a similar thing: https://github.com/bitcoin/bitcoin/pull/21055/
< luke-jr> I wonder if (re #20866) it would be a good solution to support writing to "RPC response" or some special string to get it sent over the network instead
< gribble> https://github.com/bitcoin/bitcoin/issues/20866 | Restrict RPCs that make server-side files · Issue #20866 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/04336086d3cb...372dd8da2475
< bitcoin-git> bitcoin/master 3c2e16b Cory Fields: time: add runtime sanity check
< bitcoin-git> bitcoin/master 9266f74 MarcoFalke: util: Use std::chrono for time getters
< bitcoin-git> bitcoin/master 372dd8d Wladimir J. van der Laan: Merge #21110: util: remove Boost posix_time usage from GetTime*
< bitcoin-git> [bitcoin] laanwj merged pull request #21110: util: remove Boost posix_time usage from GetTime* (master...no_boost_gettime) https://github.com/bitcoin/bitcoin/pull/21110
< jonatack> cfields: MarcoFalke: nice ^
< sipa> and then there were 9...
< queip> can we please have writable HD seeds in any form in Bitcoin Core, which makes it easier to recommend it to newbies that need easy backups on paper too? For example fixed list of 2048 english words, generate them from seed bytes (not the other way around)
< queip> actually writting out the 16(?) bytes as hex, would be not that terrible too, perhaps with some error correcting code
< sipa> with descriptor wallets that's definitely doable
< sipa> there is an issue for BIP39 support in descriptors somewhere
< sipa> but even without that, a UI could support seed phrases and convert them to descriptors
< queip> sipa: to import e.g. seed from trezor/electrum right?
< sipa> you can do that today, but you need to convert the seed phrase to a descriptor
< sipa> (also, descriptor support is experimental, so i wouldn't recommend it especially for newbies, but getting there...)
< sipa> *descriptor wallets
< queip> descriptor wallets are the one created by current versions of node, right?
< sipa> yes, and optionally
< queip> easiest things to do soon might be then an RPC to show you the bytes of your seed of given loaded wallet? and later add ECCs, and RPC to import from either format?
< sipa> you can't get the seed phrase from the master key; it's one way
< sipa> and i wouldn't say "soon" - these things should only be exposed/advertized when they work well
< sipa> there are a few things that can be worked on towards supporting that, like descriptor export, getting descriptor wallets in general less experimental, and things like bip39 support in descriptors directly
< jb55> I don't even think recommending bip39 support to 'newbies' would be a good idea. ideally there would be an easy way to connect a hardware wallet and get descriptors that way vs bip39 on your computer
< sipa> yeah
< sipa> for example
< sipa> making things both secure and user friendly is very hard
< sipa> (and even secure itself means both secure against theft and against loss, which are often in conflict with each other...)
< bitcoin-git> [bitcoin] andrewkojo opened pull request #21219: Update README.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/21219
< bitcoin-git> [bitcoin] fanquake closed pull request #21219: Update README.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/21219