2022-01-02

<bitcoin-git> bitcoin/master e9ee023 fanquake: Merge bitcoin/bitcoin#23929: doc: fix undo data filename (s/undo???.dat/re...

2022-01-01

<prayank> If you don't know how Windows shortcut works, don't know what the issue is about, confused about details and call the one who created issue misleading others or other dumb things.... Please expect a response to this non sense. Neither I am robot nor paid to manage this shit. I do this as volunteer because I like contribuing to Bitcoin.
<bitcoin-git> [bitcoin] sipa closed pull request #23935: Update README.md (master...master) https://github.com/bitcoin/bitcoin/pull/23935
<bitcoin-git> [bitcoin] hhhogannwo opened pull request #23935: Update README.md (master...master) https://github.com/bitcoin/bitcoin/pull/23935
<bitcoin-git> [bitcoin] sipa closed pull request #23934: 0.18 (master...0.18) https://github.com/bitcoin/bitcoin/pull/23934
<bitcoin-git> [bitcoin] hhhogannwo opened pull request #23934: 0.18 (master...0.18) https://github.com/bitcoin/bitcoin/pull/23934
<bitcoin-git> [bitcoin] prayank23 opened pull request #23931: create bitcoin.conf on first run with template (master...bitcoin-conf) https://github.com/bitcoin/bitcoin/pull/23931
<bitcoin-git> [bitcoin] theStack opened pull request #23929: doc: fix undo data filename (s/undo???.dat/rev???.dat/) (master...doc-fix_undo_data_filename_comment) https://github.com/bitcoin/bitcoin/pull/23929
<prayank> 2. I can work on this PR by addressing some comments from ealirer PR and new approach in which bitcoin.conf is created with some comments instead of bitcoin.conf.example
<prayank> meshcollider: jonasschnelii was not sure: https://github.com/bitcoin/bitcoin/pull/13761#issuecomment-408045138, sipa had no problem: https://github.com/bitcoin/bitcoin/pull/13761#issuecomment-408276809, laanwj NACKed it saying "Daemons should not write their own configuration file": https://github.com/bitcoin/bitcoin/pull/13761#issuecomment-408830553, A compromise was suggested by laanwj in https://github.com/bitcoin/bitcoin/pu

2021-12-31

<bitcoin-git> [bitcoin] brunoerg opened pull request #23928: test: clean up logs when there aren't perf subprocesses (master...2021-12-fix-perf-clean-up) https://github.com/bitcoin/bitcoin/pull/23928
<bitcoin-git> [gui] w0xlt opened pull request #516: Change 'Show' icon (master...change_details_icon) https://github.com/bitcoin-core/gui/pull/516
<bitcoin-git> [bitcoin] fjahr opened pull request #23927: rpc: Pruning nodes can not fetch blocks before syncing past their height (master...2021-12-prunefutureblockfetch) https://github.com/bitcoin/bitcoin/pull/23927
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23926: rpc: Quote user supplied descriptor in error msg (master...2112-rpcQuote) https://github.com/bitcoin/bitcoin/pull/23926
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/623745ca74cf...a1e04e107997
<bitcoin-git> bitcoin/master fa4c599 MarcoFalke: rpc: Quote user supplied descriptor in error msg
<bitcoin-git> bitcoin/master a1e04e1 MarcoFalke: Merge bitcoin/bitcoin#23926: rpc: Quote user supplied descriptor in error ...
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23926: rpc: Quote user supplied descriptor in error msg (master...2112-rpcQuote) https://github.com/bitcoin/bitcoin/pull/23926
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23912: Insert and bump copyright headers (master...211230-copyright) https://github.com/bitcoin/bitcoin/pull/23912
<bitcoin-git> [bitcoin] MarcoFalke pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/5d911336998e...623745ca74cf
<bitcoin-git> bitcoin/master c29105e Hennadii Stepanov: script: Fix copyright_header.py
<bitcoin-git> bitcoin/master f47dda2 Hennadii Stepanov: scripted-diff: Bump copyright headers
<bitcoin-git> bitcoin/master 1362d61 Hennadii Stepanov: scripted-diff: Insert missed copyright headers
<bitcoin-git> [bitcoin] fanquake closed pull request #23925: Update license year range to 2022 (master...patch-1) https://github.com/bitcoin/bitcoin/pull/23925
<bitcoin-git> [bitcoin] curet opened pull request #23925: Update license year range to 2022 (master...patch-1) https://github.com/bitcoin/bitcoin/pull/23925
<prayank> It's amazing how people think it's not okay for bitcoind to create its own config file with some templates in https://github.com/bitcoin/bitcoin/pull/13761
<bitcoin-git> [bitcoin] prayank23 closed pull request #23850: doc: add security recommendation about *notify config params (master...notify-security-doc) https://github.com/bitcoin/bitcoin/pull/23850

2021-12-30

<bitcoin-git> [bitcoin] dscotese opened pull request #23916: Added key import code for use with Windows. (master...patch-1) https://github.com/bitcoin/bitcoin/pull/23916
<bitcoin-git> [bitcoin] hebasto opened pull request #23912: Insert and bump copyright headers (master...211230-copyright) https://github.com/bitcoin/bitcoin/pull/23912
<bitcoin-git> [bitcoin] DrahtBot closed pull request #23903: scripted-diff: Bump copyright headers (master...2112-drahty) https://github.com/bitcoin/bitcoin/pull/23903
<bitcoin-git> [bitcoin] prusnak closed pull request #23908: Simplify macOS .dmg build by adding background.tiff to git (master...simplify-macdeploy) https://github.com/bitcoin/bitcoin/pull/23908
<bitcoin-git> [bitcoin] fanquake opened pull request #23909: build: use a static .tiff for macOS .dmg rather than generating (master...use_static_background_tiff) https://github.com/bitcoin/bitcoin/pull/23909
<bitcoin-git> [bitcoin] prusnak opened pull request #23908: Simplify macOS .dmg build by adding background.tiff to git (master...simplify-macdeploy) https://github.com/bitcoin/bitcoin/pull/23908
<bitcoin-git> [bitcoin] arnabsen1729 opened pull request #23907: tracing: utxocache tracepoints follow up for #22902 (master...2021-12-utxocache-usdt-fix) https://github.com/bitcoin/bitcoin/pull/23907
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #22815: ci: Enable the gui in the tsan build (master...210827-tsan) https://github.com/bitcoin/bitcoin/pull/22815
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f3d6a5ce772e...5d911336998e
<bitcoin-git> bitcoin/master ec7b7d4 Hennadii Stepanov: ci: Enable the gui in the TSan build
<bitcoin-git> bitcoin/master 5d91133 MarcoFalke: Merge bitcoin/bitcoin#22815: ci: Enable the gui in the tsan build
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23879: test: check ban_duration and time_remaining after setting ban (master...2021-12-test-ban_duration-time_remaining) https://github.com/bitcoin/bitcoin/pull/23879
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/de28c348a0ef...f3d6a5ce772e
<bitcoin-git> bitcoin/master da349f1 brunoerg: test: check ban_duration and time_remaining after setting ban
<bitcoin-git> bitcoin/master f3d6a5c MarcoFalke: Merge bitcoin/bitcoin#23879: test: check ban_duration and time_remaining a...
<bitcoin-git> [bitcoin] DrahtBot opened pull request #23903: scripted-diff: Bump copyright headers (master...2112-drahty) https://github.com/bitcoin/bitcoin/pull/23903
<hz> hello, Is anyone here? I wonder where I could join the bitcoin-dev meeting, just to sit in and listen
<bitcoin-git> [bitcoin] fanquake merged pull request #22814: build: Add ability to build qt in depends with -stdlib=libc++ (master...210827-libc++) https://github.com/bitcoin/bitcoin/pull/22814
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3c40355d633b...de28c348a0ef
<bitcoin-git> bitcoin/master 33796a9 Hennadii Stepanov: build: Add ability to build qt in depends with -stdlib=libc++
<bitcoin-git> bitcoin/master de28c34 fanquake: Merge bitcoin/bitcoin#22814: build: Add ability to build qt in depends wit...
<bitcoin-git> [bitcoin] fanquake merged pull request #23887: Include patches for Guix (master...bitcoin-upstream) https://github.com/bitcoin/bitcoin/pull/23887
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7e0cbc8ca4fe...3c40355d633b
<bitcoin-git> bitcoin/master 1fa2e80 Rojar Smith: Include patches for Guix
<bitcoin-git> bitcoin/master 3c40355 fanquake: Merge bitcoin/bitcoin#23887: Include patches for Guix
<bitcoin-git> [bitcoin] fanquake merged pull request #20201: build: pkg-config related cleanup (master...201020-pkg) https://github.com/bitcoin/bitcoin/pull/20201
<bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/587cbca826d7...7e0cbc8ca4fe
<bitcoin-git> bitcoin/master c236f2e Hennadii Stepanov: build: Drop redundant AC_SUBST macros
<bitcoin-git> bitcoin/master 9049812 Hennadii Stepanov: build: Drop redundant check of PKG_CHECK_MODULES presence
<bitcoin-git> bitcoin/master 7e0cbc8 fanquake: Merge bitcoin/bitcoin#20201: build: pkg-config related cleanup

2021-12-29

<bitcoin-git> [bitcoin] fanquake closed pull request #23896: Add useful test compilation documentation (master...patch-1) https://github.com/bitcoin/bitcoin/pull/23896
<bitcoin-git> [bitcoin] dhruv opened pull request #23900: rpc: nServices rpc argument for addnode (master...addnode-advertise-services) https://github.com/bitcoin/bitcoin/pull/23900
<bitcoin-git> [bitcoin] hebasto opened pull request #23897: refactor: Separate lockpoint calculate logic from CheckSequenceLocks function (master...211229-lockpoints) https://github.com/bitcoin/bitcoin/pull/23897
<bitcoin-git> [bitcoin] blueredpen opened pull request #23896: Add useful test compilation documentation (master...patch-1) https://github.com/bitcoin/bitcoin/pull/23896
<gribble> https://github.com/bitcoin/bitcoin/issues/23412 | Use notifications.dat for -*notify · Issue #23412 · bitcoin/bitcoin · GitHub
<michaelfolkson> This shouldn't be here, I'll ask questions in #bitcoin-dev or ##ctv-bip-review
<michaelfolkson> jeremyrubin: Don't worry, an intro is in the BIP. No comparison on how eltoo with APO compares to eltoo with CTV and CSFS though https://github.com/bitcoin/bips/blob/master/bip-0119.mediawiki#eltoo-with-op_checksigfromstackverify
<michaelfolkson> prayank: Plus for Taproot the BIPs doc was updated in a standalone PR https://github.com/bitcoin/bitcoin/pull/21925
<gribble> https://github.com/bitcoin/bitcoin/issues/21702 | Implement BIP-119 Validation (CheckTemplateVerify) by JeremyRubin · Pull Request #21702 · bitcoin/bitcoin · GitHub
<michaelfolkson> jeremyrubin: Can you point me to any write ups about how CTV enables eltoo channel constructions? https://bitcoin.stackexchange.com/questions/111497/are-eltoo-constructions-possible-with-the-proposed-checktemplateverify-opcode

2021-12-28

<bitcoin-git> [bitcoin] rojarsmith opened pull request #23887: Include patches for Guix (master...bitcoin-upstream) https://github.com/bitcoin/bitcoin/pull/23887
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23873: test: use MiniWallet for p2p_compactblocks.py (master...202112-test-use_MiniWallet_for_p2p_compactblocks) https://github.com/bitcoin/bitcoin/pull/23873
<bitcoin-git> bitcoin/master d2efb66 Sebastian Falbesoner: test: use MiniWallet for p2p_compactblocks.py
<bitcoin-git> bitcoin/master 587cbca MarcoFalke: Merge bitcoin/bitcoin#23873: test: use MiniWallet for p2p_compactblocks.py...
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/1292334bf5c0...587cbca826d7
<bitcoin-git> [bitcoin] fanquake merged pull request #23878: doc: Remove TODO comment in tx_verify (master...2112-docTODO) https://github.com/bitcoin/bitcoin/pull/23878
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/5d2344885bb5...1292334bf5c0
<bitcoin-git> bitcoin/master fa50d8b MarcoFalke: doc: Remove TODO comment in tx_verify
<bitcoin-git> bitcoin/master 1292334 fanquake: Merge bitcoin/bitcoin#23878: doc: Remove TODO comment in tx_verify
<bitcoin-git> [bitcoin] fanquake merged pull request #23840: doc: Fix -changetype help text (master...2112-docCT) https://github.com/bitcoin/bitcoin/pull/23840
<bitcoin-git> bitcoin/master 5d23448 fanquake: Merge bitcoin/bitcoin#23840: doc: Fix -changetype help text
<bitcoin-git> bitcoin/master fa993d0 MarcoFalke: doc: Fix -changetype help text
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/bb655a6aac93...5d2344885bb5
<bitcoin-git> [bitcoin] fanquake merged pull request #23883: doc: Remove fixed TODO from wallet/feebumper (master...2112-docWall) https://github.com/bitcoin/bitcoin/pull/23883
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/45249e70d993...bb655a6aac93
<bitcoin-git> bitcoin/master fa562fd MarcoFalke: doc: Remove fixed TODO from wallet/feebumper
<bitcoin-git> bitcoin/master bb655a6 fanquake: Merge bitcoin/bitcoin#23883: doc: Remove fixed TODO from wallet/feebumper
<bitcoin-git> [bitcoin] fanquake merged pull request #23881: doc: Remove TODO from block fuzz test (master...2112-docFuzz) https://github.com/bitcoin/bitcoin/pull/23881
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/d3582f2d3bfe...45249e70d993
<bitcoin-git> bitcoin/master fa5d0b6 MarcoFalke: doc: Remove TODO from block fuzz test
<bitcoin-git> bitcoin/master 45249e7 fanquake: Merge bitcoin/bitcoin#23881: doc: Remove TODO from block fuzz test
<bitcoin-git> [bitcoin] amadeuszpawlik closed pull request #22212: Guard `fHavePruned` to avoid potential data race (master...fix_guard_fhavepruned) https://github.com/bitcoin/bitcoin/pull/22212

2021-12-27

<bitcoin-git> [bitcoin] hebasto closed pull request #23670: build: Build test binaries in `make check`, not in `make` (master...211204-check) https://github.com/bitcoin/bitcoin/pull/23670
<bitcoin-git> [bitcoin] MarcoFalke closed pull request #21652: [WIP NOMERGE DRAFT] ci: Switch more tasks to self-hosted (master...2104-ciMoreSelf) https://github.com/bitcoin/bitcoin/pull/21652
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23884: doc: Remove outdated scriptChange TODO comment (master...2112-docWall2) https://github.com/bitcoin/bitcoin/pull/23884
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23883: doc: Remove fixed TODO from wallet/feebumper (master...2112-docWall) https://github.com/bitcoin/bitcoin/pull/23883
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23882: doc: testnet3 was not reset and is doing BIP30 checks again (master...2112-docBip30Testnet3) https://github.com/bitcoin/bitcoin/pull/23882
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23881: doc: Remove TODO from block fuzz test (master...2112-docFuzz) https://github.com/bitcoin/bitcoin/pull/23881
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23880: p2p: Serialize cmpctblock at most once in NewPoWValidBlock (master...2112-p2pAsync) https://github.com/bitcoin/bitcoin/pull/23880
<bitcoin-git> [bitcoin] brunoerg opened pull request #23879: test: check ban_duration and time_remaining after setting ban (master...2021-12-test-ban_duration-time_remaining) https://github.com/bitcoin/bitcoin/pull/23879
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23878: doc: Remove TODO comment in tx_verify (master...2112-docTODO) https://github.com/bitcoin/bitcoin/pull/23878
<bitcoin-git> [bitcoin] rojarsmith closed pull request #23874: linearize: Fix the config item name about rpc port (master...bitcoin-upstream) https://github.com/bitcoin/bitcoin/pull/23874
<bitcoin-git> [bitcoin] rojarsmith opened pull request #23874: linearize: Fix the config item name about rpc port (master...bitcoin-upstream) https://github.com/bitcoin/bitcoin/pull/23874
<bitcoin-git> [bitcoin] theStack opened pull request #23873: test: use MiniWallet for p2p_compactblocks.py (master...202112-test-use_MiniWallet_for_p2p_compactblocks) https://github.com/bitcoin/bitcoin/pull/23873
<gribble> https://github.com/bitcoin/bitcoin/issues/16899 | UTXO snapshot creation (dumptxoutset) by jamesob · Pull Request #16899 · bitcoin/bitcoin · GitHub
<bitcoin-git> [bitcoin] MarcoFalke closed pull request #23872: refactor: removed unnecessary parenthesis (master...patch-1) https://github.com/bitcoin/bitcoin/pull/23872
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23866: test: use MiniWallet for rpc_scantxoutset.py (master...202112-test-use_MiniWallet_for_rpc_scantxoutset) https://github.com/bitcoin/bitcoin/pull/23866
<bitcoin-git> [bitcoin] MarcoFalke pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/a213bd63ca0c...d3582f2d3bfe
<bitcoin-git> bitcoin/master 983ca04 Sebastian Falbesoner: test: introduce `address_to_scriptpubkey` helper
<bitcoin-git> bitcoin/master e704d4d Sebastian Falbesoner: test: introduce `getnewdestination` helper for generating various address ...
<bitcoin-git> bitcoin/master e844115 Sebastian Falbesoner: test: use MiniWallet for rpc_scantxoutset.py
<bitcoin-git> [bitcoin] fanquake merged pull request #23817: build: Fix regression introduced in PR23603 (master...211219-darwin) https://github.com/bitcoin/bitcoin/pull/23817
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/09ac6c81be9e...a213bd63ca0c
<bitcoin-git> bitcoin/master a213bd6 fanquake: Merge bitcoin/bitcoin#23817: build: Fix regression introduced in PR23603
<bitcoin-git> bitcoin/master d3b0f82 Hennadii Stepanov: build: Fix regression introduced in PR23603
<bitcoin-git> [bitcoin] fanquake merged pull request #23862: build, qt: Hardcode last modified timestamp in Qt RCC (master...211224-rcc) https://github.com/bitcoin/bitcoin/pull/23862
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/6a363720ffc1...09ac6c81be9e
<bitcoin-git> bitcoin/master 11736db Hennadii Stepanov: build, qt: Hardcode last modified timestamp in Qt RCC
<bitcoin-git> bitcoin/master 09ac6c8 fanquake: Merge bitcoin/bitcoin#23862: build, qt: Hardcode last modified timestamp i...
<bitcoin-git> [bitcoin] fanquake closed pull request #23869: Update copyright year to 2022 (master...master) https://github.com/bitcoin/bitcoin/pull/23869
<bitcoin-git> [bitcoin] fanquake closed pull request #23868: Prettified `README.md` (master...master) https://github.com/bitcoin/bitcoin/pull/23868
<bitcoin-git> [bitcoin] Selmaleh28 closed pull request #23870: Bazinga (master...Bazinga) https://github.com/bitcoin/bitcoin/pull/23870
<bitcoin-git> [bitcoin] Selmaleh28 opened pull request #23870: Bazinga (master...Bazinga) https://github.com/bitcoin/bitcoin/pull/23870
<bitcoin-git> [bitcoin] sandakersmann opened pull request #23869: Update copyright year to 2022 (master...master) https://github.com/bitcoin/bitcoin/pull/23869

2021-12-26

<bitcoin-git> [bitcoin] retr0cube opened pull request #23868: Prettified `README.md` (master...master) https://github.com/bitcoin/bitcoin/pull/23868
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23864: doc: Drop outdated note (master...211225-riscv) https://github.com/bitcoin/bitcoin/pull/23864
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/9bec5b80a0f3...6a363720ffc1
<bitcoin-git> bitcoin/master 94a7f09 Hennadii Stepanov: doc: Drop outdated note
<bitcoin-git> bitcoin/master 6a36372 MarcoFalke: Merge bitcoin/bitcoin#23864: doc: Drop outdated note
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23858: test: Fix rpc_scantxoutset intermittent issue (master...2112-testFix) https://github.com/bitcoin/bitcoin/pull/23858
<bitcoin-git> bitcoin/master fa0ac9d MarcoFalke: test: Fix rpc_scantxoutset intermittent issue
<bitcoin-git> bitcoin/master fafe4de MarcoFalke: test: Fix pep8 of touched file
<bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/010686198322...9bec5b80a0f3
<bitcoin-git> bitcoin/master 9bec5b8 MarcoFalke: Merge bitcoin/bitcoin#23858: test: Fix rpc_scantxoutset intermittent issue...
<bitcoin-git> [bitcoin] luke-jr reopened pull request #16228: Bugfix: make check: Only run tests that were compiled (master...bugfix_raii_check_fail) https://github.com/bitcoin/bitcoin/pull/16228
<bitcoin-git> [bitcoin] theStack opened pull request #23866: test: use MiniWallet for rpc_scantxoutset.py (master...202112-test-use_MiniWallet_for_rpc_scantxoutset) https://github.com/bitcoin/bitcoin/pull/23866

2021-12-25

<bitcoin-git> [bitcoin] achow101 closed pull request #23865: Update README.md (master...master) https://github.com/bitcoin/bitcoin/pull/23865
<bitcoin-git> [bitcoin] pzxnv7 opened pull request #23865: Update README.md (master...master) https://github.com/bitcoin/bitcoin/pull/23865
<Kiminuo> Hi, would anyone mind having a look at https://github.com/bitcoin/bitcoin/pull/23320? It had one ACK(ish) here https://github.com/bitcoin/bitcoin/pull/23320#issuecomment-976552487.
<prayank> gmax should consider becoming a maintainer again, maybe things improve and if he has enough time. anyways thanks for all the info. I am going out now and will try things in few hours. It would be great if it helps cj in anyway. And leaks: bitcoin is many like its the easiest thing to become a privacy researcher in bitcoin.
<jeremyrubin> i should probably write these details up somewhere and link to it from the BIP. GMAX has also discussed it in #bitcoin-wizards
<jeremyrubin> prayank: sapio depends on rust-bitcoin and rust-miniscript. neither have released Taproot support yet -- once they do, i need to rebase Sapio patches and then do a little work to integrate it, then it will be fine (but I need to check and make sure things like PSBTs still work fine).
<gribble> https://github.com/bitcoin/bitcoin/issues/21702 | Implement BIP-119 Validation (CheckTemplateVerify) by JeremyRubin · Pull Request #21702 · bitcoin/bitcoin · GitHub
<prayank> jeremyrubin: Compiling #21702 branch. BIP 119 mentioned about coinjoin. I want to test this with a setup of 2-3 regtest nodes. I don't understand the code written here: https://github.com/bitcoin/bips/blob/master/bip-0119.mediawiki#detailed-specification however I can test things based on the description that "participants agree on a single output which pays all participants". Do you have any other suggestion to help me in this
<bitcoin-git> [bitcoin] hebasto opened pull request #23864: doc: Drop outdated note (master...211225-riscv) https://github.com/bitcoin/bitcoin/pull/23864
<bitcoin-git> [bitcoin] fanquake merged pull request #23859: test: Add missing suppressions for crypto_diff_fuzz_chacha20.cpp (master...2112-testSup) https://github.com/bitcoin/bitcoin/pull/23859
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/369978686e15...010686198322
<bitcoin-git> bitcoin/master 0106861 fanquake: Merge bitcoin/bitcoin#23859: test: Add missing suppressions for crypto_dif...
<bitcoin-git> bitcoin/master faaf9da MarcoFalke: test: Add missing suppressions for crypto_diff_fuzz_chacha20.cpp

2021-12-24

<bitcoin-git> [bitcoin] hebasto opened pull request #23862: build, qt: Hardcode last modified timestamp in Qt RCC (master...211224-rcc) https://github.com/bitcoin/bitcoin/pull/23862
<bitcoin-git> [bitcoin] hebasto opened pull request #23861: build: Update Guix build free space requirements (master...211224-kib) https://github.com/bitcoin/bitcoin/pull/23861
<bitcoin-git> [bitcoin] MarcoFalke closed pull request #19831: test: Check that decoderawtransaction heuristic may fail (master...2008-testDecodeRawT) https://github.com/bitcoin/bitcoin/pull/19831
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23859: test: Add missing suppressions for crypto_diff_fuzz_chacha20.cpp (master...2112-testSup) https://github.com/bitcoin/bitcoin/pull/23859
<bitcoin-git> [bitcoin] hebasto reopened pull request #23609: build: Enables reduced exports by default for macOS host (master...211126-reduce) https://github.com/bitcoin/bitcoin/pull/23609
<bitcoin-git> [bitcoin] MarcoFalke opened pull request #23858: test: Fix rpc_scantxoutset intermittent issue (master...2112-testFix) https://github.com/bitcoin/bitcoin/pull/23858
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23835: test: check for invalid listtransactions RPC parameters (master...202112-test-rpc-check_invalid_listtransactions_parameters) https://github.com/bitcoin/bitcoin/pull/23835
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/dada92fed2e8...369978686e15
<bitcoin-git> bitcoin/master 3699786 MarcoFalke: Merge bitcoin/bitcoin#23835: test: check for invalid listtransactions RPC ...
<bitcoin-git> bitcoin/master c27bba9 Sebastian Falbesoner: test: check for invalid listtransactions RPC parameters
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #22875: util: Fix Racy ParseOpCode function initialization (master...threadsafe-fix) https://github.com/bitcoin/bitcoin/pull/22875
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7551ae81e1a9...dada92fed2e8
<bitcoin-git> bitcoin/master 7b481f0 Jeremy Rubin: Fix Racy ParseOpCode function initialization
<bitcoin-git> bitcoin/master dada92f MarcoFalke: Merge bitcoin/bitcoin#22875: util: Fix Racy ParseOpCode function initializ...
<bitcoin-git> [bitcoin] fanquake opened pull request #23857: fs: consistently use fsbridge:: for ifstream / ofstream (master...always_use_fsbridge) https://github.com/bitcoin/bitcoin/pull/23857

2021-12-23

<dongcarl> bitcoin-git is back alive
<bitcoin-git> [bitcoin] dongcarl opened pull request #23855: Post-"Chainstate loading sequence coalescence" fixups (master...2021-12-kernel-lib-v3-fixups) https://github.com/bitcoin/bitcoin/pull/23855
<larryruane> `-loadblock` does have the problem that blocks must be sorted, as explained here: https://github.com/bitcoin/bitcoin/pull/20331 ... and usually they're not sorted
<laanwj> MarcoFalke: no, i don't think that's possible using build-in functionality afaik, what i usually do is use https://github.com/bitcoin-core/bitcoin-maintainer-tools#fastcopy-chaindata to make a copy of the chain and chainstate, then prune that, then copy the pruned copy to the smaller medium
<laanwj> bitcoin-git reporting should be fixed (same problem again, https://github.com/gkrizek/ghi/issues/13)
<jeremyrubin> FYI in response to some private feedback i've clarified a few points in the BIP around alternative vault designs (the original one is outdated from my latest thinking) and activation stuff https://github.com/bitcoin/bips/pull/1257

2021-12-22

<gribble> https://github.com/bitcoin/bitcoin/issues/21726 | Improve Indices on pruned nodes via prune blockers by fjahr · Pull Request #21726 · bitcoin/bitcoin · GitHub
<dongcarl> Hey all, wondering if anyone has an idea on how best to test this functionality: https://github.com/bitcoin/bitcoin/blob/63b5dfac21613b984803347bfe81454da667016e/src/validation.cpp#L2225-L2230
<gribble> https://github.com/bitcoin/bitcoin/issues/23027 | Bugfix: Skip tests for tools not being built by luke-jr · Pull Request #23027 · bitcoin/bitcoin · GitHub
<sipa> ha, i didn't know this: cirrus has a big bitcoin core logo on their site under "Trusted by Open Source": https://cirrus-ci.org/
<gribble> https://github.com/bitcoin/bitcoin/issues/23213 | rest: Return error when header count is not integral by MarcoFalke · Pull Request #23213 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/23227 | bitcoin-tx: Avoid treating integer overflow as OP_0 by MarcoFalke · Pull Request #23227 · bitcoin/bitcoin · GitHub
<MarcoFalke> for bitcoin-tx it was broken either way (with or without overflow), already fixed in #23227
<MarcoFalke> Yeah, not consensus. Could be used by bitcoin-util or some RPC function.
<sipa> RPC, P2P, multiprocess, UI... they're all distinct ways in which Bitcoin Core interacts with other processes/systems; ... and they all have different needs, and use different tools to accomplish that. Just because one uses a particular technology under the hood doesn't mean it's easy to make another one use the same technology.
<sipa> If you're using Linux, and using bitcoin-qt, Qt is likely for some part at least using a UNIX domain socket to communicate with the X graphics driver. So if your goal was "I want my Bitcoin Core to use UNIX domain sockets!!!", then you probably already have it.
<sipa> But it doesn't (and can't) provide the Bitcoin Core RPC interface. It's used to provide the multiprocess interface.
<michaelfolkson> bitcoin-node in multiprocess uses UNIX sockets https://github.com/bitcoin/bitcoin/pull/19460

2021-12-21

<jeremyrubin> Marcofalke: in my view we want 3 projects to happen: 1 Bitcoin kernel to make consensus boundaries clear, 2 multiprocess to separate out things that don't need to share memory (most important imo would be having the network threads only able to message pass safely to core) and then 3 minimal node where bits of functionality in core are replaced
<laanwj> instead of removing everything that isn't required i like the libbitcoinkernel direction more, isolate the consensus code, it allows you to use that within anything else if you don't agree with what bitcoin core offers
<gribble> https://github.com/bitcoin/bitcoin/issues/22850 | net: Make m_mempool optional in PeerManager by sriramdvt · Pull Request #22850 · bitcoin/bitcoin · GitHub

2021-12-20

<achow101> bitcoin-git ded?
<sipa> Oh, I was talking about https://github.com/bitcoin/bitcoin/pull/9919.
<gribble> https://github.com/bitcoin/bitcoin/issues/9979 | p2p: Bare minimum to support UNIX sockets by laanwj · Pull Request #9979 · bitcoin/bitcoin · GitHub
<sipa> I'd very much like to see at least a UNIX domain socket added, but it seems earlier attempts to do that were complicated by lack of unix support on the client-side of libevent, which is used in bitcoin-cli.
<jeremyrubin> overall i think it'd just be good if we only exposed Unix Sockets and had everything else be done via proxy. We can get all the local services working as we want them do (e/g bitcoin-cli) and industrial users should already be proxying. it's just attack surface with little benefit
<jeremyrubin> also bad would be being able to detect a user is running bitcoin software while web browsing, even if no request is fired. but that applies to jsonrpc too
<bitcoin-git> [bitcoin] ryanofsky opened pull request #23828: scripted-diff: Rename libbitcoin_server.a to libbitcoin_node.a (master...pr/libnode) https://github.com/bitcoin/bitcoin/pull/23828
<bitcoin-git> [bitcoin] mzumsande opened pull request #23826: test: Make AddrMan unit tests use public interface, extend coverage (master...202112_addrman_unit_tests_1) https://github.com/bitcoin/bitcoin/pull/23826
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23796: test: check that pruneblockchain RPC fails for future block or timestamp (master...202112-test-add_pruneblockchain_test_coverage) https://github.com/bitcoin/bitcoin/pull/23796
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/20aea4945d80...9ac064d245f0
<bitcoin-git> bitcoin/master 140a49c Sebastian Falbesoner: test: check that pruneblockchain RPC fails for future block or timestamp
<bitcoin-git> bitcoin/master 9ac064d MarcoFalke: Merge bitcoin/bitcoin#23796: test: check that pruneblockchain RPC fails fo...
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23797: ci: Use Cirrus "greedy" flag to use idle CPU time when available (master...cirrus_greedy) https://github.com/bitcoin/bitcoin/pull/23797
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3ec8f9f123bb...20aea4945d80
<bitcoin-git> bitcoin/master cfeb194 Luke Dashjr: ci: Use Cirrus "greedy" flag to use idle CPU time when available
<bitcoin-git> bitcoin/master 20aea49 MarcoFalke: Merge bitcoin/bitcoin#23797: ci: Use Cirrus "greedy" flag to use idle CPU ...
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23801: Refactor: Change time variable type from int64_t to std::chrono::seconds in net_processing.cpp (master...time_in_seconds_refactor) https://github.com/bitcoin/bitcoin/pull/23801
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/23afc5f47ba1...3ec8f9f123bb
<bitcoin-git> bitcoin/master 92082ea Shashwat: Change time variable type to std::chrono::seconds in src/net_processing.cp...
<bitcoin-git> bitcoin/master 3ec8f9f MarcoFalke: Merge bitcoin/bitcoin#23801: Refactor: Change time variable type from int6...
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23711: docs: RBF policy and mempool limit exemptions (master...2021-12-rbf-docs) https://github.com/bitcoin/bitcoin/pull/23711
<bitcoin-git> bitcoin/master 919ae8b glozow: [doc] current rbf policy
<bitcoin-git> bitcoin/master 82858ba glozow: [doc] CPFP carve out and single-conflict RBF exemption
<bitcoin-git> bitcoin/master 1fd49eb glozow: [doc] clarify RBF difference from BIP125
<bitcoin-git> [bitcoin] MarcoFalke pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/d1dc6b895fe1...23afc5f47ba1
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23780: refactor, test: update `addrman_tests.cpp` to use output from `AddrMan::Good()` (master...josibake-addrman-test-refactors) https://github.com/bitcoin/bitcoin/pull/23780
<bitcoin-git> [bitcoin] MarcoFalke pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/1c41fb918155...d1dc6b895fe1
<bitcoin-git> bitcoin/master e281fcc josibake: refactor: addrman_noevict test
<bitcoin-git> bitcoin/master 5a64dc0 josibake: refactor: addrman_evictionworks test
<bitcoin-git> bitcoin/master 8bdd924 josibake: refactor: addrman_selecttriedcollisions test
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23812: test: fix intermittent failures in p2p_timeouts.py (master...202112_p2p_timeout_fix) https://github.com/bitcoin/bitcoin/pull/23812
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/70d6a09f5ccd...1c41fb918155
<bitcoin-git> bitcoin/master 1c41fb9 MarcoFalke: Merge bitcoin/bitcoin#23812: test: fix intermittent failures in p2p_timeou...
<bitcoin-git> bitcoin/master 0a1b6fa Martin Zumsande: test: fix intermittent timeouts in p2p_timeouts.py
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #17631: Expose block filters over REST (master...2019-11-filter-rest) https://github.com/bitcoin/bitcoin/pull/17631
<bitcoin-git> bitcoin/master ef7c822 Matt Corallo: Expose block filters over REST.
<bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/8eeb4e997b06...70d6a09f5ccd
<bitcoin-git> bitcoin/master 2b64fa3 Matt Corallo: Update REST docs with new accessors
<bitcoin-git> bitcoin/master 70d6a09 MarcoFalke: Merge bitcoin/bitcoin#17631: Expose block filters over REST
<bitcoin-git> [bitcoin] fanquake merged pull request #23798: contrib: add achow101 to trusted keys (master...wallet-maintainer-key) https://github.com/bitcoin/bitcoin/pull/23798
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/d05be150ca23...8eeb4e997b06
<bitcoin-git> bitcoin/master 33b5112 Andrew Chow: contrib: add achow101 to trusted keys
<bitcoin-git> bitcoin/master 8eeb4e9 fanquake: Merge bitcoin/bitcoin#23798: contrib: add achow101 to trusted keys
<ademan[m]> #bitcoin is probably a good place too
<jb55> Guest2020: try asking at https://bitcoin.stackexchange.com, this place is more for dev stuff

2021-12-19

<bitcoin-git> [bitcoin] jb55 opened pull request #23819: tracing/block_connected: don't serialize block hash twice (master...faster_block_connected_usdt) https://github.com/bitcoin/bitcoin/pull/23819
<bitcoin-git> [bitcoin] fjahr closed pull request #18000: [WIP] Index for UTXO Set Statistics (master...utxo-stats-index-rebase) https://github.com/bitcoin/bitcoin/pull/18000
<bitcoin-git> [bitcoin] sipa closed pull request #19503: Add parameter feature to serialization and use it for CAddress (master...202007_ser_options) https://github.com/bitcoin/bitcoin/pull/19503
<bitcoin-git> [bitcoin] hebasto closed pull request #23609: build: Enables reduced exports by default for macOS host (master...211126-reduce) https://github.com/bitcoin/bitcoin/pull/23609
<bitcoin-git> [bitcoin] hebasto opened pull request #23817: 211219 darwin (master...211219-darwin) https://github.com/bitcoin/bitcoin/pull/23817
<bitcoin-git> [bitcoin] fanquake merged pull request #23811: scripts: match on `EXE_FORMATS` rather than name string (master...sec_check_match_type) https://github.com/bitcoin/bitcoin/pull/23811
<bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/c006ab29ceec...d05be150ca23
<bitcoin-git> bitcoin/master feee308 fanquake: scripts: match on exe type over str in security-check.py
<bitcoin-git> bitcoin/master d05be15 fanquake: Merge bitcoin/bitcoin#23811: scripts: match on `EXE_FORMATS` rather than n...
<bitcoin-git> bitcoin/master d1711a4 fanquake: scripts: match on exe type over str in symbol-check.py
<bitcoin-git> [bitcoin] luke-jr closed pull request #20550: RPC: Tolerate invalid rpcauth options (master...soften_rpcauth2) https://github.com/bitcoin/bitcoin/pull/20550
<bitcoin-git> [bitcoin] luke-jr closed pull request #19790: Flag when blocks have had their scripts checked instead of skipped (master...blkindex_scriptschecked_flag) https://github.com/bitcoin/bitcoin/pull/19790

2021-12-18

<bitcoin-git> [bitcoin] darosior closed pull request #23815: wallet: watchonly support for Miniscript descriptors (master...miniscript_wallet_watchonly) https://github.com/bitcoin/bitcoin/pull/23815
<bitcoin-git> [bitcoin] darosior opened pull request #23815: wallet: watchonly support for Miniscript descriptors (master...miniscript_wallet_watchonly) https://github.com/bitcoin/bitcoin/pull/23815
<bitcoin-git> [bitcoin] laanwj merged pull request #23219: p2p, refactor: tidy up LookupSubNet() (master...LookupSubNet-todo-fix) https://github.com/bitcoin/bitcoin/pull/23219
<bitcoin-git> [bitcoin] laanwj pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/cb27d60f966a...c006ab29ceec
<bitcoin-git> bitcoin/master c44c201 Vasil Dimov: p2p, refactor: drop unused DNSLookupFn param in LookupSubnet()
<bitcoin-git> bitcoin/master f0c9e68 Jon Atack: p2p, refactor: tidy up LookupSubNet()
<bitcoin-git> bitcoin/master c006ab2 W. J. van der Laan: Merge bitcoin/bitcoin#23219: p2p, refactor: tidy up LookupSubNet()
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23814: ci: Bump the macOS image to the latest Monterey (master...211218-monterey) https://github.com/bitcoin/bitcoin/pull/23814
<bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/1220af5e6d10...cb27d60f966a
<bitcoin-git> bitcoin/master 836d027 Hennadii Stepanov: ci: Bump the macOS image to the latest Monterey
<bitcoin-git> bitcoin/master 34fd25e Hennadii Stepanov: ci: Use system sqlite in native macOS task
<bitcoin-git> bitcoin/master cb27d60 MarcoFalke: Merge bitcoin/bitcoin#23814: ci: Bump the macOS image to the latest Monter...
<bitcoin-git> [bitcoin] laanwj merged pull request #23781: test: Fix `system_tests/run_command` on Windows (master...211215-bptest) https://github.com/bitcoin/bitcoin/pull/23781
<bitcoin-git> [bitcoin] laanwj pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/98a2ddcd6ed0...1220af5e6d10
<bitcoin-git> bitcoin/master 507c009 Hennadii Stepanov: test: Fix "echo" subtest in the system_tests for Windows
<bitcoin-git> bitcoin/master 0aad33d Hennadii Stepanov: test: Fix "false" subtest in system_tests for Windows
<bitcoin-git> bitcoin/master fb1b059 Hennadii Stepanov: test: Fix "non-zero exit code" subtest in system_tests for Windows
<bitcoin-git> [bitcoin] hebasto opened pull request #23814: ci: Bump the macOS image to the latest Monterey (master...211218-monterey) https://github.com/bitcoin/bitcoin/pull/23814
<bitcoin-git> [bitcoin] fjahr opened pull request #23813: Add test and docs for getblockfrompeer (master...2021-12-getblockfrompeer-follow-up) https://github.com/bitcoin/bitcoin/pull/23813
<bitcoin-git> [bitcoin] mzumsande opened pull request #23812: test: fix intermittent failures in p2p_timeout.py (master...202112_p2p_timeout_fix) https://github.com/bitcoin/bitcoin/pull/23812
<bitcoin-git> [bitcoin] fanquake opened pull request #23811: scripts: match on `EXE_FORMATS` rather than name string (master...sec_check_match_type) https://github.com/bitcoin/bitcoin/pull/23811
<laanwj> PSA: as it was clear that fanquake has overwhelming support i've made him an admin of the bitcoin and bitcoin-core organizations
<bitcoin-git> [bitcoin] MarcoFalke merged pull request #23806: fuzz: follow up for #22704 (master...follow-up-diff-fuzz-chacha20) https://github.com/bitcoin/bitcoin/pull/23806
<bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c06cda3e48e9...98a2ddcd6ed0
<bitcoin-git> bitcoin/master 8f79831 stratospher: Refactor the chacha20 differential fuzz test
<bitcoin-git> bitcoin/master 98a2ddc MarcoFalke: Merge bitcoin/bitcoin#23806: fuzz: follow up for #22704
<bitcoin-git> [bitcoin] fanquake merged pull request #23383: Update libsecp256k1 subtree to current master (master...202110_secp256k1) https://github.com/bitcoin/bitcoin/pull/23383
<bitcoin-git> bitcoin/master 86dbc4d Pieter Wuille: Squashed 'src/secp256k1/' changes from be8d9c262f..0559fc6e41
<bitcoin-git> bitcoin/master a1f76cd Pieter Wuille: Remove --disable-openssl-tests for libsecp256k1 configure
<bitcoin-git> [bitcoin] fanquake pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/97b2fc03f666...c06cda3e48e9
<bitcoin-git> bitcoin/master dff0596 Pieter Wuille: Update secp256k1 subtree to latest upstream master
<bitcoin-git> [bitcoin] PastaPastaPasta opened pull request #23810: refactor: destroy all C-style casts; use modern C++ casts, enforce via `-Wold-style-cast` (master...c-style-cast) https://github.com/bitcoin/bitcoin/pull/23810
<bitcoin-git> [bitcoin] fanquake merged pull request #23803: ci: Disable s390x gui tests for now (master...2112-ciGUIs390x) https://github.com/bitcoin/bitcoin/pull/23803
<bitcoin-git> bitcoin/master fa6a548 MarcoFalke: ci: Disable s390x gui tests for now
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/14ba286556fa...97b2fc03f666
<bitcoin-git> bitcoin/master 97b2fc0 fanquake: Merge bitcoin/bitcoin#23803: ci: Disable s390x gui tests for now
<bitcoin-git> [bitcoin] jb55 opened pull request #23809: tracing: reduce instructions generated by net:inbound_message tracepoint (master...conntype-enum-usdt) https://github.com/bitcoin/bitcoin/pull/23809

2021-12-17

<jb55> btw satellite doesn't need to be a bitcoin fork right? couldn't it just be a block-from-space decoder + call submitblock? or is there more to it?
<sipa> But Bitcoin Core works fine with tor itself, and tor itself also works with browsers.
<sipa> Yes, he needs a way to communicate with Bitcoin Core. The altnet stuff goes in the direction of providing that functionality. As I said, I conceptually agree with that. I still don't see the point of node-tor, but if there is more functionality for interacting with separate networks, and someone makes it work with node-tor, sure...
<sipa> That's all very reasonable, but I don't think that has much to do with Bitcoin Core.
<prayank> sipa: Reason IIUC he doesn't like the way Tor project uses Tor protocol and its flaws etc. but wants to make a better to for different projects and bitcoin is one of them
<jb55> so basically a way to feed network messages in and out as plaintext, turn bitcoin into a pure function?
<prayank> sipa: what I understood based on his emails. he likes browsers, js and world around it. so webtorrents kind of things for bitcoin nodes. He is from france, he is good with his project but unable to express or get funding. IDK just wanted to share it here once so that people like you who know better would be able to see if it helps bitcoin at some point. Everything is moving to browsers tbh and this is not about altcoins. I work
<sipa> But again, while I conceptually agree with making it easier to connect bitcoin nodes in more different ways with eachother, I don't understand the point of node-tor.
<sipa> I don't think that's at all what he's talking about. He's saying that the communication between bitcoin core and node-tor (if it were to use node-tor rather than actual tor) would not use SOCKS.
<prayank> hmm tbh I am not the best person to comment anything on this. I liked the idea and his follow up on mailing list. What I understand is: we do bitcoin nodes from browsers similar to one PoC by bcoin but its not safe for production. And this guy wants to make it interesting by using tor protocol for everything. I have issues with JS and WebRTC but if it can help bitcoin nodes usage I dont mind and maybe not full nodes or somethin
<sipa> I conceptually like having more mechanisms for connecting bitcoin nodes though, so if there were a viable network like that to use as an additional one - sure why not.
<sipa> I don't understanding anything about node-tor, or why you'd want to use a separate tor network for just bitcoin traffic. That completely defeats the purpose.
<prayank> Any thoughts on this? especially people who understand p2p, tor etc. https://github.com/bitcoin/bitcoin/pull/18988#issuecomment-646564853
<bitcoin-git> [bitcoin] laanwj merged pull request #23695: p2p: Always serialize local timestamp for version msg (master...2112-p2pTime) https://github.com/bitcoin/bitcoin/pull/23695
<bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/784a21d35466...14ba286556fa
<bitcoin-git> bitcoin/master 14ba286 W. J. van der Laan: Merge bitcoin/bitcoin#23695: p2p: Always serialize local timestamp for ver...
<bitcoin-git> bitcoin/master fa1dc9b MarcoFalke: p2p: Always serialize local timestamp for version msg
<bitcoin-git> [bitcoin] laanwj merged pull request #22283: build: Replace $(AT) with .SILENT (master...replace_AT_with_dotsilence) https://github.com/bitcoin/bitcoin/pull/22283
<bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/4ad59042b359...784a21d35466
<bitcoin-git> bitcoin/master 8494dca Dmitry Goncharov: Replace $(AT) with .SILENCE.