{ "founder": "wumpus", "channel": "#bitcoin-core-dev", "network": "freenode", "id": "1da9b67788824fd38d91685d05701426", "name": "#bitcoin-core-dev", "chair": "wumpus", "chairs": [ "wumpus" ], "nicks": { "wumpus": 46, "jonasschnelli_": 1, "jonasschnelli": 22, "kanzure": 1, "ajonas": 10, "hebasto": 1, "achow101": 2, "amiti": 13, "jnewbery": 52, "fjahr": 2, "sipa": 43, "promag": 1, "jonatack": 6, "gribble": 14, "ariard": 17, "elichai2": 1, "MarcoFalke": 42, "sdaftuar": 22, "luke-jr": 22, "lightlike": 1, "darosior": 1, "phantomcircuit": 1 }, "start_time": "2021-03-04T19:00:55+00:00", "end_time": "2021-03-04T20:03:27+00:00", "active": false, "original_topic": "Bitcoin Core development discussion and commit log | Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt / http://gnusha.org/bitcoin-core-dev/proposedwalletmeetingtopics.txt", "current_topic": "Developer survey (ajonas)", "messages": [ { "id": "d0ddc473209245608995fd0b29edb858", "sender": "wumpus", "payload": "#startmeeting", "action": false, "timestamp": "2021-03-04T19:00:55+00:00" }, { "id": "93e549fc54384acda0da72d1125e6918", "sender": "jonasschnelli_", "payload": "(again)", "action": false, "timestamp": "2021-03-04T19:01:01+00:00" }, { "id": "1f068d0232b64feabe9d9de49183efa8", "sender": "jonasschnelli", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:10+00:00" }, { "id": "98b3d87315bf4e01893a10f06d32bbcd", "sender": "kanzure", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:11+00:00" }, { "id": "23d8644882004cb19c75d7c87c1726ef", "sender": "ajonas", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:13+00:00" }, { "id": "089b930e691548be90b5905ffe6664af", "sender": "hebasto", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:17+00:00" }, { "id": "bd0ec9196edf4bcabde6b8f0caccb408", "sender": "achow101", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:17+00:00" }, { "id": "4e0d53208a5f4925ba595262a3545400", "sender": "amiti", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:18+00:00" }, { "id": "90d312cd9303492986291737bcb9cc72", "sender": "jnewbery", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:20+00:00" }, { "id": "d3288b992b234510b7be7945eb4271be", "sender": "wumpus", "payload": "one proposed meeting topic: blocksonly mode and transaction relay (jnewbery)", "action": false, "timestamp": "2021-03-04T19:01:44+00:00" }, { "id": "412920e2fdbc41d798565ca9575efc91", "sender": "fjahr", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:52+00:00" }, { "id": "8225c0bd671346b1884ebfa7e014281b", "sender": "wumpus", "payload": "#bitcoin-core-dev Meeting: achow101 aj amiti ariard bluematt cfields Chris_Stewart_5 digi_james dongcarl elichai2 emilengler fanquake fjahr gleb glozow gmaxwell gwillen hebasto instagibbs jamesob jb55 jeremyrubin jl2012 jnewbery jonasschnelli jonatack jtimon kallewoof kanzure kvaciral lightlike luke-jr maaku marcofalke meshcollider michagogo moneyball morcos nehan NicolasDorier paveljanik", "action": false, "timestamp": "2021-03-04T19:01:55+00:00" }, { "id": "d9fbddd1f99c4b99b5d8ab2712d8f49a", "sender": "wumpus", "payload": "petertodd phantomcircuit promag provoostenator ryanofsky sdaftuar sipa vasild wumpus", "action": false, "timestamp": "2021-03-04T19:01:58+00:00" }, { "id": "2ad9c0d17bba441390351103105bd8ec", "sender": "sipa", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:01:58+00:00" }, { "id": "a665033785ac498c8ae4f54f5fb1137d", "sender": "promag", "payload": "o/", "action": false, "timestamp": "2021-03-04T19:02:35+00:00" }, { "id": "22bf1b8266b94edd9a86c0e92cb9f53b", "sender": "wumpus", "payload": "any last-minute topics?", "action": false, "timestamp": "2021-03-04T19:02:39+00:00" }, { "id": "ac1959a3de4d43f0be0d6d65fb4df3f0", "sender": "jonatack", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:02:45+00:00" }, { "id": "428b8da9aab944c4aad7f2216a4e0545", "sender": "ajonas", "payload": "wumpus: I just have a brief mention of the developer survey", "action": false, "timestamp": "2021-03-04T19:02:49+00:00" }, { "id": "7e46934265e84d28a44f2a14c30f7503", "sender": "wumpus", "payload": "ajonas: great!", "action": false, "timestamp": "2021-03-04T19:03:04+00:00" }, { "id": "81bfe455014847a4869a7a5d38baf778", "sender": "wumpus", "payload": "#topic High priority for review", "action": false, "timestamp": "2021-03-04T19:03:43+00:00" }, { "id": "707926108b5046c48ceae8cb5d14b5a3", "sender": "jonasschnelli", "payload": "I added #20664 to the hiprio list", "action": false, "timestamp": "2021-03-04T19:03:57+00:00" }, { "id": "63b41d6b6e8f4fdb8839cc3a8a252e28", "sender": "wumpus", "payload": "https://github.com/bitcoin/bitcoin/projects/8 9 blockers, 2 chasing concept ACK", "action": false, "timestamp": "2021-03-04T19:04:00+00:00" }, { "id": "90ee9703efd9454581b1dd480452ea8c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/20664 | Add scanblocks RPC call by jonasschnelli \u00c3\u0082\u00c2\u00b7 Pull Request #20664 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:04:00+00:00" }, { "id": "0f2166ab2f0a428f88d996646241d9ad", "sender": "wumpus", "payload": "does anyone else want to add or remove anything ?", "action": false, "timestamp": "2021-03-04T19:04:40+00:00" }, { "id": "ffb791b5534b4d6f93eee3d40b07e944", "sender": "jonatack", "payload": "maybe add #20197 please, if the list isn't too long", "action": false, "timestamp": "2021-03-04T19:04:47+00:00" }, { "id": "fcd609de7aa44522a035a2aa6816e1d8", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/20197 | p2p: protect onions in AttemptToEvictConnection(), add eviction protection test coverage by jonatack \u00c3\u0082\u00c2\u00b7 Pull Request #20197 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:04:50+00:00" }, { "id": "7ec8e35b88fe462ab6a4b779a2e472c1", "sender": "wumpus", "payload": "jonatack: added!", "action": false, "timestamp": "2021-03-04T19:05:24+00:00" }, { "id": "72213cdbd8d3449cb6a14c5adcb7b852", "sender": "jonatack", "payload": "wumpus: ty!", "action": false, "timestamp": "2021-03-04T19:05:33+00:00" }, { "id": "5d2864fcebc44a6cbbb477ae85b8a967", "sender": "wumpus", "payload": "is there anything (almost) ready for merge?", "action": false, "timestamp": "2021-03-04T19:06:46+00:00" }, { "id": "e6a86ff4d8ae45e4988dc86dbff2cd09", "sender": "ariard", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:07:45+00:00" }, { "id": "3a229ffbf4884da6a128cbd0dac1e79c", "sender": "achow101", "payload": "#20536 has 3 acks", "action": false, "timestamp": "2021-03-04T19:07:52+00:00" }, { "id": "2b51ec0687eb4fc0bc70078d630dde85", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/20536 | wallet: Error with \"Transaction too large\" if the funded tx will end up being too large after signing by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #20536 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:07:53+00:00" }, { "id": "ce3141812a84418cae8d1cc296ed6281", "sender": "elichai2", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:07:55+00:00" }, { "id": "2bcdc30af6654666ba9e2cf8be92c5d3", "sender": "MarcoFalke", "payload": "hi", "action": false, "timestamp": "2021-03-04T19:07:58+00:00" }, { "id": "2c698e18e4fb4eebb8c6884c0c6ae2c3", "sender": "ajonas", "payload": "There are a few old tests that have got some new review... nothing major though", "action": false, "timestamp": "2021-03-04T19:07:59+00:00" }, { "id": "153f8937741e461aaba2f6e00e7149b7", "sender": "wumpus", "payload": "achow101: good to know", "action": false, "timestamp": "2021-03-04T19:08:04+00:00" }, { "id": "0df52d68de4f45e4a1575bd274f4e8c8", "sender": "wumpus", "payload": "i guess that's all for this topic, let's go to next one then", "action": false, "timestamp": "2021-03-04T19:08:53+00:00" }, { "id": "b0c8009e156f4aa7a2b0894d4711e396", "sender": "ajonas", "payload": "#18795, #19393, #14604", "action": false, "timestamp": "2021-03-04T19:08:53+00:00" }, { "id": "bead1dc524714e3ebd90129b1224b4fe", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18795 | Test: wallet issue with orphaned rewards by domob1812 \u00c3\u0082\u00c2\u00b7 Pull Request #18795 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:08:55+00:00" }, { "id": "e581e4b6089f4fdebf77fdcab3514fd1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/19393 | test: Add more tests for orphan tx handling by hebasto \u00c3\u0082\u00c2\u00b7 Pull Request #19393 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:08:57+00:00" }, { "id": "abb86a8799f5453199f55e81cae49f61", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14604 | tests: Add test and refactor feature_block.py by sanket1729 \u00c3\u0082\u00c2\u00b7 Pull Request #14604 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:08:59+00:00" }, { "id": "ffd2751d97ce4027bdfdef22c1aeec64", "sender": "wumpus", "payload": "ajonas: thank you", "action": false, "timestamp": "2021-03-04T19:09:13+00:00" }, { "id": "59bc62a531c84c4fa0adcb249a06f4dc", "sender": "wumpus", "payload": "#topic Blocksonly mode and transaction relay (jnewbery)", "action": false, "timestamp": "2021-03-04T19:09:39+00:00" }, { "id": "bc111754d70d47f6aab4679bec694cd6", "sender": "jnewbery", "payload": "hello!", "action": false, "timestamp": "2021-03-04T19:09:59+00:00" }, { "id": "39082691d2004020802159256b0169e7", "sender": "jnewbery", "payload": "There is a `-blocksonly` option in Bitcoin Core, which disables transaction relay for a node.", "action": false, "timestamp": "2021-03-04T19:10:03+00:00" }, { "id": "51239de0117e408b8b5d734668edee15", "sender": "jnewbery", "payload": "It does this by setting fRelay=false on the version message on all of its connections.", "action": false, "timestamp": "2021-03-04T19:10:14+00:00" }, { "id": "e3e70e032dec4e778c52a436b0ec17fa", "sender": "jnewbery", "payload": "This was added in PR #6993 and was made a public documented option #15990 (https://github.com/bitcoin/bitcoin/blob/master/doc/reduce-traffic.md#4-turn-off-transaction-relay--blocksonly)", "action": false, "timestamp": "2021-03-04T19:10:24+00:00" }, { "id": "56eff8f0f7104a76a4c773ac71134a63", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/6993 | Add -blocksonly option by pstratem \u00c3\u0082\u00c2\u00b7 Pull Request #6993 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:10:26+00:00" }, { "id": "4b6cb3127bb043dba46e3ee042e57dd1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/15990 | Add tests and documentation for blocksonly by MarcoFalke \u00c3\u0082\u00c2\u00b7 Pull Request #15990 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:10:29+00:00" }, { "id": "f6411ae15b3c4ad88f6dc824cb9c3c6d", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/4 | Export/Import wallet in a human readable, future-proof format \u00c3\u0082\u00c2\u00b7 Issue #4 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:10:31+00:00" }, { "id": "3e1db45aa31f46df9543c9116e3cad53", "sender": "jnewbery", "payload": "One strange (and potentially footgun-ish aspect) of -blocksonly is that if a transaction is submitted to the node over RPC, or sent by a whitelisted peer, then the node will relay that transaction to all its peers.", "action": false, "timestamp": "2021-03-04T19:10:40+00:00" }, { "id": "2f5c1b33a8964703b86ec053ada7fb05", "sender": "jnewbery", "payload": "This makes it very obvious that the node is the origin of the transaction, since it doesn't relay any other transaction!", "action": false, "timestamp": "2021-03-04T19:10:52+00:00" }, { "id": "032eb2196bba47fe9470b78b81dc55b5", "sender": "jnewbery", "payload": "Are there legitimate use cases for this behaviour?", "action": false, "timestamp": "2021-03-04T19:11:02+00:00" }, { "id": "31835bfa884c4c1f927b6bfd2d534250", "sender": "ariard", "payload": "I would say emergency broadcast, you don't care about the privacy", "action": false, "timestamp": "2021-03-04T19:11:22+00:00" }, { "id": "a7257c67c6e14790880130133c23d474", "sender": "sipa", "payload": "fRelay=false is also set in other cases, right?", "action": false, "timestamp": "2021-03-04T19:11:29+00:00" }, { "id": "3339c09796494973b9fda7ae2e3f47f3", "sender": "sipa", "payload": "for blocks relay connections, even if you're not in blocksonly mode?", "action": false, "timestamp": "2021-03-04T19:11:46+00:00" }, { "id": "219af063092b43f59b86d86fe4df8b10", "sender": "ariard", "payload": "like when your main node is upgrading/power outage/...", "action": false, "timestamp": "2021-03-04T19:11:50+00:00" }, { "id": "70f41c72be184dcc816ef61112a967b5", "sender": "jnewbery", "payload": "by Bitcoin Core? We set fRelay=false for -blocksonly mode, or for the block-relay-only connections, even if you're not in -blocksonly mode", "action": false, "timestamp": "2021-03-04T19:12:01+00:00" }, { "id": "844ef13671f34479b85242c671e7840b", "sender": "sipa", "payload": "oh, nvm, i was confusing things; this isn't relevant", "action": false, "timestamp": "2021-03-04T19:12:33+00:00" }, { "id": "19abe72cdf4442e5888c56f2f0002d23", "sender": "jnewbery", "payload": "bloom filter clients also set fRelay=false when connecting to supress tx relay until they've loaded the filter", "action": false, "timestamp": "2021-03-04T19:12:49+00:00" }, { "id": "bcbe438e42e54b4bb3f557dca9c2aead", "sender": "sipa", "payload": "i'm not sure what the right behavior is", "action": false, "timestamp": "2021-03-04T19:13:21+00:00" }, { "id": "6c1164555a814031a3482896bd53ad84", "sender": "sipa", "payload": "it's clearly a privacy leak, but i think it would also be incorrect to just not relay", "action": false, "timestamp": "2021-03-04T19:13:35+00:00" }, { "id": "1a3982f9c1e54c1eb09951c4a37325fd", "sender": "wumpus", "payload": "maybe reject it by default but add a 'force' flag to override, i dunno", "action": false, "timestamp": "2021-03-04T19:13:41+00:00" }, { "id": "f90e8347ef9e477f8d6ddd77725b45bf", "sender": "sipa", "payload": "there is an expectation that a transaction is relayed if you call sendrawtransaction", "action": false, "timestamp": "2021-03-04T19:13:50+00:00" }, { "id": "df5c3109e1f14a36a629b7697ac15130", "sender": "fjahr", "payload": "I was just going to suggest a force flag too", "action": false, "timestamp": "2021-03-04T19:14:00+00:00" }, { "id": "0f302dea28fa4eb29e7b965aea7907ac", "sender": "wumpus", "payload": "yea failing silently would be the worst of both worlds", "action": false, "timestamp": "2021-03-04T19:14:03+00:00" }, { "id": "0dbff0e622dc4837b8f31967fe814cbe", "sender": "ariard", "payload": "yeah force flag sounds good", "action": false, "timestamp": "2021-03-04T19:14:05+00:00" }, { "id": "dab9dccf89774a6dba0ae5cf427db84c", "sender": "jonasschnelli", "payload": "+1 for the force flag", "action": false, "timestamp": "2021-03-04T19:14:08+00:00" }, { "id": "f7c9a34b6c114bd086b921c588140ca8", "sender": "jnewbery", "payload": "always fail sendrawtransaction if -blocksonly is set?", "action": false, "timestamp": "2021-03-04T19:14:12+00:00" }, { "id": "83839852f0f84d209a577d42c5e751eb", "sender": "jonasschnelli", "payload": "jnewbery: seems a bit to hand-holdish", "action": false, "timestamp": "2021-03-04T19:14:28+00:00" }, { "id": "4e49cd915bb9479b8690763d538ea36d", "sender": "MarcoFalke", "payload": "Seems a bit too hand-holdish to add a force flag for this", "action": false, "timestamp": "2021-03-04T19:14:50+00:00" }, { "id": "57fbc287f0c54f688a70d92a480ec9b5", "sender": "MarcoFalke", "payload": "The privacy leak is well documented in the option help", "action": false, "timestamp": "2021-03-04T19:15:04+00:00" }, { "id": "c0d114c883c24e83aa62be959f951a50", "sender": "jnewbery", "payload": "a force flag seems the worst of all worlds", "action": false, "timestamp": "2021-03-04T19:15:13+00:00" }, { "id": "a6150ee63143417486043047a781817b", "sender": "sdaftuar", "payload": "can't someone use bitcoin-qt in blocksonly mode? i'm not understanding hte force flag in that scenario", "action": false, "timestamp": "2021-03-04T19:15:15+00:00" }, { "id": "c8a7f98eb8304d269d51ddff276e55e3", "sender": "sdaftuar", "payload": "some warnign popup i guess?", "action": false, "timestamp": "2021-03-04T19:15:24+00:00" }, { "id": "b25b4243747d485590880cc724d91102", "sender": "jonasschnelli", "payload": "The GUI could have a warning, yes.", "action": false, "timestamp": "2021-03-04T19:15:34+00:00" }, { "id": "58af2a27467b4bac802fd7bf05c5df6f", "sender": "jnewbery", "payload": "oh, one thing I should add: -blocksonly will soft-set -walletbroadcast to false", "action": false, "timestamp": "2021-03-04T19:15:42+00:00" }, { "id": "19e8bcce8bbe423295176e9e4e9e36ac", "sender": "jonasschnelli", "payload": "Which is more or less the UX adaption of a \"force\" CLI flag", "action": false, "timestamp": "2021-03-04T19:15:51+00:00" }, { "id": "29b92fa235354d5799a2a76d6e2ce823", "sender": "luke-jr", "payload": "MarcoFalke: it fits well with your force-past-policy PR", "action": false, "timestamp": "2021-03-04T19:15:52+00:00" }, { "id": "c23b9aa748f243ef85bea53d12cc466e", "sender": "wumpus", "payload": "it doesn't imply walletbroadca.. right", "action": false, "timestamp": "2021-03-04T19:15:54+00:00" }, { "id": "dd9c1581714b42eb9ca131a363f67942", "sender": "wumpus", "payload": "so we did think of that", "action": false, "timestamp": "2021-03-04T19:16:17+00:00" }, { "id": "afed87d0bdb34f12b6b87b0f39683a8a", "sender": "jonatack", "payload": "sendrawtransaction help states the issue clearly", "action": false, "timestamp": "2021-03-04T19:16:21+00:00" }, { "id": "8c2019f850504402a7a7cb25a12ece09", "sender": "jonasschnelli", "payload": "sendrawtransaction respectes walletbroadcast?", "action": false, "timestamp": "2021-03-04T19:16:22+00:00" }, { "id": "6ed44acb5d2a4b569a8cab8eb5835321", "sender": "MarcoFalke", "payload": "as jnewbery points out the wallet should disable the broadcast with blocksonly", "action": false, "timestamp": "2021-03-04T19:16:23+00:00" }, { "id": "b824768ec0ad44b4b1e1b7fc44214950", "sender": "jnewbery", "payload": "I don't expect users to understand the privacy implications of this. Putting an option in the GUI seems overly complex", "action": false, "timestamp": "2021-03-04T19:16:29+00:00" }, { "id": "c60c5f07ecb54757ac46555d0ad6db79", "sender": "wumpus", "payload": "i also remember this came up before", "action": false, "timestamp": "2021-03-04T19:16:37+00:00" }, { "id": "d2c785aba299482bacaf152f0301415e", "sender": "MarcoFalke", "payload": "but the rpc is separate", "action": false, "timestamp": "2021-03-04T19:16:38+00:00" }, { "id": "a14055888a974b03ae72df2423522b1f", "sender": "sdaftuar", "payload": "so are we only talking about changing behavior in the case that -blocksonly=true and -walletbroadcast=true?", "action": false, "timestamp": "2021-03-04T19:16:47+00:00" }, { "id": "653275b4b01b496894c5be3b6b0b87ef", "sender": "jnewbery", "payload": "sendrawtransaction is not a wallet rpc, so should have no interaction with walletrebroadcast", "action": false, "timestamp": "2021-03-04T19:16:59+00:00" }, { "id": "1c4c5eae5d4e4e2b85e40b420a172d06", "sender": "sdaftuar", "payload": "if the user has set that, it seems clear waht the intent is", "action": false, "timestamp": "2021-03-04T19:17:07+00:00" }, { "id": "d69e6be222d14e0e8fd0c8cd8926e58e", "sender": "luke-jr", "payload": "sdaftuar: the intent of sendrawtransaction is also clear and contradicts it", "action": false, "timestamp": "2021-03-04T19:17:28+00:00" }, { "id": "5fc47d75045f4c718da402d54cfd8ed1", "sender": "wumpus", "payload": "sdaftuar: +1", "action": false, "timestamp": "2021-03-04T19:17:28+00:00" }, { "id": "b7aa9086f51c4952a702663bbcb3944a", "sender": "MarcoFalke", "payload": "luke-jr: It doesn't contradict. It overwrites for a specific tx", "action": false, "timestamp": "2021-03-04T19:17:55+00:00" }, { "id": "0c7b78cde83148169519b97650f95271", "sender": "jonasschnelli", "payload": "if you don't have transaction relay, doesn't the wallet always need manual fee settings (problematic either)?", "action": false, "timestamp": "2021-03-04T19:18:04+00:00" }, { "id": "037b4c9728a947bdbc7df7ee5126e7c9", "sender": "MarcoFalke", "payload": "jonasschnelli: This is also documented", "action": false, "timestamp": "2021-03-04T19:18:18+00:00" }, { "id": "2279d491000b43de99591f109ceaa93b", "sender": "jnewbery", "payload": "jonasschnelli: indeed", "action": false, "timestamp": "2021-03-04T19:18:21+00:00" }, { "id": "648a1f9b5fec4293b0e94e6ec6becd56", "sender": "MarcoFalke", "payload": "https://github.com/bitcoin/bitcoin/blob/master/doc/reduce-traffic.md#4-turn-off-transaction-relay--blocksonly", "action": false, "timestamp": "2021-03-04T19:18:25+00:00" }, { "id": "38d38cfb12ba40668db89307fa602654", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/4 | Export/Import wallet in a human readable, future-proof format \u00c3\u0082\u00c2\u00b7 Issue #4 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:18:27+00:00" }, { "id": "027b5e0bc4a742c69fd64da50108b0c7", "sender": "MarcoFalke", "payload": "If we assume that no one reads the docs, what is the point of adding it in the first place.", "action": false, "timestamp": "2021-03-04T19:18:48+00:00" }, { "id": "7d9e19953248478e9dadb53e9a7c5e7d", "sender": "jonasschnelli", "payload": "yes. that is a valid point.", "action": false, "timestamp": "2021-03-04T19:19:02+00:00" }, { "id": "64e9dc1bb9f54af092318247a8aac366", "sender": "jonasschnelli", "payload": "Maybe we should expect people read the docs", "action": false, "timestamp": "2021-03-04T19:19:11+00:00" }, { "id": "114ded4335aa4ff4a6121a375f80f809", "sender": "jnewbery", "payload": "My baseline assumption is that users don't read docs", "action": false, "timestamp": "2021-03-04T19:19:12+00:00" }, { "id": "df3357579bad4a148e016ab441b4f503", "sender": "amiti", "payload": "MarcoFalke: I think its not boolean? I'd expect some users to read docs, but not all", "action": false, "timestamp": "2021-03-04T19:19:25+00:00" }, { "id": "a00022c2078d4cf2a5ce6d7e38c266f7", "sender": "sipa", "payload": "i'm a bit confused about the context... this is only about the sendrawtransaction RPC, not about wallet sending?", "action": false, "timestamp": "2021-03-04T19:19:35+00:00" }, { "id": "30aafb61fe8248e58bb162929c8f215f", "sender": "jnewbery", "payload": "and I don't think documenting bad behaviour is better than removing bad behaviour", "action": false, "timestamp": "2021-03-04T19:19:43+00:00" }, { "id": "8262d09cabeb4da781576bfa22988883", "sender": "jonasschnelli", "payload": "jnewbery: if so, how do they even find out about the blocksonly mode? :-)", "action": false, "timestamp": "2021-03-04T19:19:45+00:00" }, { "id": "e582434243844ee2805e7b5436d46f8d", "sender": "luke-jr", "payload": "jonasschnelli: reading *some* docs but not others! XD", "action": false, "timestamp": "2021-03-04T19:20:00+00:00" }, { "id": "a084d686177c4c04af5546ba4e4c34ea", "sender": "MarcoFalke", "payload": "sipa: I think this is a general discussion about the version message fRelay field", "action": false, "timestamp": "2021-03-04T19:20:04+00:00" }, { "id": "d4bd6df8cdb540ceb7d5635806f029ef", "sender": "jnewbery", "payload": "jonasschnelli: because someone suggests that they enable it on a twitter thread to save themselves some bandwidth?", "action": false, "timestamp": "2021-03-04T19:20:25+00:00" }, { "id": "d5b739fe00704a90a8a0b32fccb76122", "sender": "jonasschnelli", "payload": "yes... that", "action": false, "timestamp": "2021-03-04T19:20:42+00:00" }, { "id": "98087d23e8ce4e2690a81d2262364cec", "sender": "sipa", "payload": "what specifically are we discussing changing?", "action": false, "timestamp": "2021-03-04T19:20:55+00:00" }, { "id": "522a019b2f0b4c25abae6d08f56670af", "sender": "sdaftuar", "payload": "the followup question to this, i believe, is whether it's reasonable for our software to ignore transactions from a peer that has set fRelay=false -- do i have that right jnewbery?", "action": false, "timestamp": "2021-03-04T19:20:57+00:00" }, { "id": "92026af232784f59ad4384ed88648840", "sender": "jnewbery", "payload": "sdaftuar: that's the wider context, yes", "action": false, "timestamp": "2021-03-04T19:21:19+00:00" }, { "id": "9e3778ac540b4c878f611196ecb44b3c", "sender": "sdaftuar", "payload": "so this question is to establish whether our current relay behavior is sort of reasonable at all", "action": false, "timestamp": "2021-03-04T19:21:33+00:00" }, { "id": "dd30d89418cf4f2d81f31e10993cca4e", "sender": "jonasschnelli", "payload": "Aren't there any valid use-cases that use blocksonly & wallet broadcast (like a private network, designated connection)?", "action": false, "timestamp": "2021-03-04T19:21:39+00:00" }, { "id": "619d6bf63cdf4fd2bc638e11caf96d3b", "sender": "MarcoFalke", "payload": "jnewbery: There is the use case where the node(s) with blocksonly only connect to trusted nodes. It seems too broad to call the feature bad", "action": false, "timestamp": "2021-03-04T19:21:40+00:00" }, { "id": "ef46e3c63b0941648d6683ac4b54e620", "sender": "jonasschnelli", "payload": "Agree with MarcoFalke", "action": false, "timestamp": "2021-03-04T19:22:02+00:00" }, { "id": "f425fee4a09449ba94a1f256ab3aa391", "sender": "sipa", "payload": "MarcoFalke: i'd expect people to set whitelisting in that case, though (or they'll experience more broken behavior)", "action": false, "timestamp": "2021-03-04T19:22:17+00:00" }, { "id": "e144a327784c48aeb4ad7fab54c9a6e2", "sender": "sipa", "payload": "like rebroadcasting won't work", "action": false, "timestamp": "2021-03-04T19:22:18+00:00" }, { "id": "d96de729e0584edb8b9f1eb0927bbab9", "sender": "jnewbery", "payload": "I don't understand this use case", "action": false, "timestamp": "2021-03-04T19:22:22+00:00" }, { "id": "3c71cc1bddd3482491ec6ace497efbfb", "sender": "sdaftuar", "payload": "sipa: i think rebroadcasting would work fine? it'll relay to your node in the same circumstances that your node would succeed in relaying it to their peers, no?", "action": false, "timestamp": "2021-03-04T19:23:13+00:00" }, { "id": "96a4c6a999914ed9a574b816ca96aada", "sender": "luke-jr", "payload": "jnewbery: maybe the user just doesn't care about privacy?", "action": false, "timestamp": "2021-03-04T19:23:14+00:00" }, { "id": "6628de9fd39a44b89596e38f9aa5e467", "sender": "jnewbery", "payload": "luke-jr: that's fair, but is that a use case we want to support?", "action": false, "timestamp": "2021-03-04T19:23:49+00:00" }, { "id": "0cf0196235194367852fff13141ac9cc", "sender": "luke-jr", "payload": "\u00c3\u00a2\u00c2\u0080\u00c2\u00a6", "action": false, "timestamp": "2021-03-04T19:23:55+00:00" }, { "id": "38099931a7bc47f79ab8329bb9b07f03", "sender": "MarcoFalke", "payload": "sipa: sendrawtrasnaction is designed to do a rebroadcast every time you call it", "action": false, "timestamp": "2021-03-04T19:24:05+00:00" }, { "id": "b88ded9a9ab3429c994b5343ba19f6ba", "sender": "sipa", "payload": "MarcoFalke: right, but wallet rebroadcasting won't work, because alreadyhave logic will think the (only) peer already has it", "action": false, "timestamp": "2021-03-04T19:24:36+00:00" }, { "id": "199f13b4227448278cdbffec98708738", "sender": "sipa", "payload": "not alreadyhave; inventoryknown", "action": false, "timestamp": "2021-03-04T19:24:51+00:00" }, { "id": "3a3cfb1350dd4b66a87311f26ed2bff8", "sender": "MarcoFalke", "payload": "permission flags can't override the inv-getdata filter either, IIRC", "action": false, "timestamp": "2021-03-04T19:24:51+00:00" }, { "id": "56dbbd14b6734d998fd0a21a61f259f0", "sender": "sipa", "payload": "hmm, fair; even if you announce again, the peer won't fetch it again", "action": false, "timestamp": "2021-03-04T19:25:25+00:00" }, { "id": "a72ae5355ede493594879f15010350a4", "sender": "MarcoFalke", "payload": "your inventoryknown is the other peer's alreadyhave ;)", "action": false, "timestamp": "2021-03-04T19:25:26+00:00" }, { "id": "dda42021ca53408590fdef2d76e89e4b", "sender": "sipa", "payload": "i'm confused now", "action": false, "timestamp": "2021-03-04T19:25:29+00:00" }, { "id": "b5e2350596224a6d96dc8460677f9bdb", "sender": "jnewbery", "payload": "transaction relay is a public network function, and it should be designed in such a way that is optimal for users connecting to public networks", "action": false, "timestamp": "2021-03-04T19:25:40+00:00" }, { "id": "4b74e2bf50ec4796a0e5984e1b7cd2b0", "sender": "sipa", "payload": "i thought that was the reason we introduced whitelisting in the first place", "action": false, "timestamp": "2021-03-04T19:25:46+00:00" }, { "id": "fb7707a037494f62a85947f1e9ac73db", "sender": "luke-jr", "payload": "jnewbery: I never liked the idea of telling users what they can and can't do. And if Core is going to do that, at least the protocol needs to be neutral.", "action": false, "timestamp": "2021-03-04T19:25:49+00:00" }, { "id": "d7c18c9ffdd8445baf6dc0f15453e0ab", "sender": "sdaftuar", "payload": "oh! we never clear filterInventoryKnown? that seems potentially problematic", "action": false, "timestamp": "2021-03-04T19:26:10+00:00" }, { "id": "30fe1122542a493cbf90a3047552449e", "sender": "amiti", "payload": "I find it unexpected that the default behavior of -blocksonly mode is to disable wallet broadcast, but if you submit via sendrawtransaction RPC the node will still relay your transaction", "action": false, "timestamp": "2021-03-04T19:26:35+00:00" }, { "id": "a329742f6b734f0587d72cdbca50ab81", "sender": "sipa", "payload": "doesn't look like it", "action": false, "timestamp": "2021-03-04T19:26:36+00:00" }, { "id": "dddc1cc94ba44901bdb1556a00a24cdb", "sender": "sipa", "payload": "amiti: that's inconsistent for sure", "action": false, "timestamp": "2021-03-04T19:26:47+00:00" }, { "id": "feee8cbd306240499b14f9c1c237978b", "sender": "sipa", "payload": "but i don't really know what to do about it", "action": false, "timestamp": "2021-03-04T19:26:56+00:00" }, { "id": "9e07ab662442415eb617e3bdf2d06247", "sender": "jnewbery", "payload": "I'm still curious about the supposed use cases. jonasschnelli and MarcoFalke thing there are use cases, but I don't understand them", "action": false, "timestamp": "2021-03-04T19:27:33+00:00" }, { "id": "cc9139df78db4561917dc693276e265b", "sender": "ariard", "payload": "actually you can be willingly to not participate in tx-relay to mask your connection graph", "action": false, "timestamp": "2021-03-04T19:27:33+00:00" }, { "id": "78c54f33adda4810a43f05694088c494", "sender": "ariard", "payload": "and only broardcat when you really need it", "action": false, "timestamp": "2021-03-04T19:27:41+00:00" }, { "id": "dd40d10c4e8a4f18959b6fafc060293e", "sender": "amiti", "payload": "sipa: what's the issue with just giving the user an error message if they attempt sendrawtransaction?", "action": false, "timestamp": "2021-03-04T19:27:44+00:00" }, { "id": "6cae8bc4070e4350b8daaaaea9b2c22d", "sender": "amiti", "payload": "maybe not desirable, but one way to make it consistent", "action": false, "timestamp": "2021-03-04T19:28:04+00:00" }, { "id": "5f40c7df82574366a3f047d917996f7c", "sender": "jonasschnelli", "payload": "jnewbery: Assume someone runs a blockonly node at home (bandwith) but connects it to a trusted peer over wireguard", "action": false, "timestamp": "2021-03-04T19:28:23+00:00" }, { "id": "912d10cbf6774425bc0e9537968b321d", "sender": "MarcoFalke", "payload": "sendrawtransaction is explicitly called. Wallet broadcast is implicit, so I think the current behavior makes sense", "action": false, "timestamp": "2021-03-04T19:28:38+00:00" }, { "id": "9f4fbba339464960808eeb667d69302a", "sender": "MarcoFalke", "payload": "(It is possible to enable wallet broadcast explicitly as well)", "action": false, "timestamp": "2021-03-04T19:28:53+00:00" }, { "id": "aa45128159724b77b6831edfc6764da2", "sender": "amiti", "payload": "MarcoFalke: fair", "action": false, "timestamp": "2021-03-04T19:29:00+00:00" }, { "id": "703925e725a14972ac366c2e97075926", "sender": "jnewbery", "payload": "jonasschnelli: that'd associate all of your transactions to your node", "action": false, "timestamp": "2021-03-04T19:29:01+00:00" }, { "id": "6e8372cb18a245c6adee9c6256b86c12", "sender": "luke-jr", "payload": "bandwidth vs privacy is a tradeoff. Users may prefer bandwidth.", "action": false, "timestamp": "2021-03-04T19:29:07+00:00" }, { "id": "979aa8663de343f598f3c0dbc8be229d", "sender": "sdaftuar", "payload": "jnewbery: does that matter to tor users?", "action": false, "timestamp": "2021-03-04T19:29:13+00:00" }, { "id": "5acc129bbc314bf7afff06cf2e8bf512", "sender": "MarcoFalke", "payload": "jnewbery: Which is no problem because you control the other node", "action": false, "timestamp": "2021-03-04T19:29:16+00:00" }, { "id": "a29df54f42f24c5b8bcd10782dd2a70c", "sender": "lightlike", "payload": "Am I right to assume that this issue with sendrawtransaction doesn't apply to regular blocks-only connections in normal mode - only in full blocksonly mode?", "action": false, "timestamp": "2021-03-04T19:29:28+00:00" }, { "id": "23e44501ad1c433cab74d462da039edd", "sender": "ariard", "payload": "but even if you don't control the other node, you prefer the transaction propagating over the privacy gain", "action": false, "timestamp": "2021-03-04T19:29:39+00:00" }, { "id": "1f10e1418e26498d890cfedcc75846bb", "sender": "jnewbery", "payload": "MarcoFalke. Presumably the home node is relaying the transaction on to the network?", "action": false, "timestamp": "2021-03-04T19:29:41+00:00" }, { "id": "80ae57e71e214afe9653ef86b5517417", "sender": "luke-jr", "payload": "wonders if Tor nodes can/should make a dedicated connection just for broadcast", "action": true, "timestamp": "2021-03-04T19:29:42+00:00" }, { "id": "810451b18cd74d37bcf731475feb29d3", "sender": "amiti", "payload": "lightlike: yes", "action": false, "timestamp": "2021-03-04T19:29:49+00:00" }, { "id": "c889ec79b77e40de96daf63665550394", "sender": "ariard", "payload": "you *might* prefer (e.g time-sensitive txn)", "action": false, "timestamp": "2021-03-04T19:29:55+00:00" }, { "id": "69d66238ba2f4cd490f2637db2463046", "sender": "MarcoFalke", "payload": "sdaftuar: If you don't control the tor node, I'd say yes. (The other peer will learn your whole wallet contents over time)", "action": false, "timestamp": "2021-03-04T19:30:11+00:00" }, { "id": "6e5fb701bc3e42a8a759c7c8ccc8db30", "sender": "sipa", "payload": "so sendrawtransaction does not relay over fRelay=false connections in general, only in -blocksonly mode?", "action": false, "timestamp": "2021-03-04T19:30:17+00:00" }, { "id": "19a762c28af9464aa68c0cc9de1bbdae", "sender": "jnewbery", "payload": "sipa: I belive that is correct", "action": false, "timestamp": "2021-03-04T19:30:37+00:00" }, { "id": "0f13043f8d014937b452aafbbfa4fdd1", "sender": "sipa", "payload": "curious", "action": false, "timestamp": "2021-03-04T19:30:45+00:00" }, { "id": "19c5a0b415ba4117916f558890166a39", "sender": "MarcoFalke", "payload": "sendrawtransaction will send it out to all peers that have send you a fRelay=true", "action": false, "timestamp": "2021-03-04T19:30:54+00:00" }, { "id": "73a6ced819de46c98ff57561e8ef11f9", "sender": "sdaftuar", "payload": "fRelay=false isn't a \"category\" of connection, really", "action": false, "timestamp": "2021-03-04T19:30:56+00:00" }, { "id": "f6252e1efe1e4df9bdca449172b3970a", "sender": "luke-jr", "payload": "jnewbery: in that scenario, the home node talks exclusively to your dedicated server somewhere over VPN", "action": false, "timestamp": "2021-03-04T19:30:56+00:00" }, { "id": "70995c10743a41f3adcf13b89cc0dd26", "sender": "MarcoFalke", "payload": "sendrawtransaction doesn't take into account what fRelay you sent to the peer", "action": false, "timestamp": "2021-03-04T19:31:22+00:00" }, { "id": "01e95393fa93484c864698f956a6f7f7", "sender": "sdaftuar", "payload": "we have full-relay-outbound connections, which might actually be fRelay=false if we're in -blocksonly mode", "action": false, "timestamp": "2021-03-04T19:31:23+00:00" }, { "id": "2bb874ef6cbc4cc985a94b2ab988990b", "sender": "jnewbery", "payload": "luke-jr: I don't think that's what jonasschnelli was describing", "action": false, "timestamp": "2021-03-04T19:31:29+00:00" }, { "id": "9f87ed5508d94cbb968ec092d0006c6e", "sender": "sipa", "payload": "ah, i see", "action": false, "timestamp": "2021-03-04T19:31:31+00:00" }, { "id": "ca88c6241b284f7e95afba73aa9c01d3", "sender": "sdaftuar", "payload": "but we treat them as our tx relay peers anyway (which is correct, in the sense that fRelay=false only instructs our peer to not relay to us, not the other way around)", "action": false, "timestamp": "2021-03-04T19:31:42+00:00" }, { "id": "d1c7a6696c774186979999fc5aba5f60", "sender": "sipa", "payload": "jnewbery: that's how i interpret jonasschnelli's scenario as well", "action": false, "timestamp": "2021-03-04T19:31:47+00:00" }, { "id": "6e8de8f24130492c8d47cc778b12717d", "sender": "jonasschnelli", "payload": "yes. m2", "action": false, "timestamp": "2021-03-04T19:31:53+00:00" }, { "id": "446b1eb8c7ee4cdc9db5af93da3524f5", "sender": "sipa", "payload": "(only one connection, blocksonly to a trusted peer; you don't care about privacy w.r.t. that peer)", "action": false, "timestamp": "2021-03-04T19:32:08+00:00" }, { "id": "46c010f08dde4627953a2b3bcded8743", "sender": "sipa", "payload": "i think in general \"i don't care about privacy w.r.t. this peer\" is something to be captured by whitelisting", "action": false, "timestamp": "2021-03-04T19:33:00+00:00" }, { "id": "d8f267ea28044cec8dad35eb1ec5f7bd", "sender": "sipa", "payload": "but of course... who knows what people read", "action": false, "timestamp": "2021-03-04T19:33:08+00:00" }, { "id": "6f0bc5d9619a4598b9c4b570d6cc2601", "sender": "luke-jr", "payload": "hmm", "action": false, "timestamp": "2021-03-04T19:33:14+00:00" }, { "id": "046a3085526e4b338598df5da1a0a040", "sender": "MarcoFalke", "payload": "Another use case might be to create a larger gap from your hot wallet to the live p2p network (multiprocess isn't a thing yet)", "action": false, "timestamp": "2021-03-04T19:33:28+00:00" }, { "id": "d6a6b250aa014d9f9d5ff3d3d92382ca", "sender": "ariard", "payload": "sipa: whitelisting is static, you might willingly to bypass \"i don't care about privacy w.r.t this peer\" for a one-shot broadcast", "action": false, "timestamp": "2021-03-04T19:33:43+00:00" }, { "id": "b52046608db046f6bd0f8441cef5fe9e", "sender": "MarcoFalke", "payload": "sipa: It is called permission flags now ;)", "action": false, "timestamp": "2021-03-04T19:33:53+00:00" }, { "id": "7f6cc2f346884218af65505f9c5a3f5b", "sender": "sipa", "payload": "MarcoFalke: right, thanks", "action": false, "timestamp": "2021-03-04T19:33:59+00:00" }, { "id": "5ce94d7314d74aafa3d782de68d9238d", "sender": "jnewbery", "payload": "ah, thank you. I was misunderstanding. I think if you control both ends of the connection, then it'd make more sense for the home node to set feefilter=max to the trusted peer", "action": false, "timestamp": "2021-03-04T19:34:43+00:00" }, { "id": "e83f3ded7b1c4a379d87cc11e90d3de0", "sender": "sipa", "payload": "ariard: that seems more like something you'd want for tor connections (and as a specific way of broadcasting transactions)", "action": false, "timestamp": "2021-03-04T19:34:46+00:00" }, { "id": "766e73ba74284aaeae86b43567c799db", "sender": "MarcoFalke", "payload": "And permission flags don't work for outbound connections (yet)", "action": false, "timestamp": "2021-03-04T19:35:02+00:00" }, { "id": "371cfd8df4794def9ad8c0bc814d7cc8", "sender": "luke-jr", "payload": "MarcoFalke: have a PR for that for a year+ now :P", "action": false, "timestamp": "2021-03-04T19:35:39+00:00" }, { "id": "c4e314d96f614eefa13b076c66319faf", "sender": "wumpus", "payload": "what is holding it up?", "action": false, "timestamp": "2021-03-04T19:35:56+00:00" }, { "id": "eefab47a94d84994b820c1bc7a537919", "sender": "luke-jr", "payload": "lack of review last I checked", "action": false, "timestamp": "2021-03-04T19:36:02+00:00" }, { "id": "efca263427a34597ab733e0c906e4722", "sender": "MarcoFalke", "payload": "either that or \"Needs rebase\"", "action": false, "timestamp": "2021-03-04T19:36:09+00:00" }, { "id": "3dff01275ce84fad969ca951cfc255a6", "sender": "sipa", "payload": "pr number?", "action": false, "timestamp": "2021-03-04T19:36:22+00:00" }, { "id": "331d37f9631a4e1189e4572393ccc6f6", "sender": "sipa", "payload": "(mentioning it here may help :p)", "action": false, "timestamp": "2021-03-04T19:36:29+00:00" }, { "id": "a0bce75c0d5249328a7880b68686a0fe", "sender": "wumpus", "payload": "yes", "action": false, "timestamp": "2021-03-04T19:36:35+00:00" }, { "id": "820705ebdfa54806b63f2fe6185f7a71", "sender": "luke-jr", "payload": "#17167", "action": false, "timestamp": "2021-03-04T19:36:36+00:00" }, { "id": "8fdfa2ebe0854395bf0fd433efd3e3c4", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/17167 | Allow whitelisting outgoing connections by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #17167 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:36:38+00:00" }, { "id": "6aee005340044ec9a74996fbce52e208", "sender": "luke-jr", "payload": "looks like it needs rebase at the moment", "action": false, "timestamp": "2021-03-04T19:36:46+00:00" }, { "id": "5368819dddfe4f749b19dd24bdb448f2", "sender": "MarcoFalke", "payload": "jnewbery: Sending feefilter=max to the trusted peer could work", "action": false, "timestamp": "2021-03-04T19:36:56+00:00" }, { "id": "210c28fc2e854325ad7862cdbd4ea157", "sender": "wumpus", "payload": "i have always liked the idea of being able to whitelist ^D^D permission outgoing connections", "action": false, "timestamp": "2021-03-04T19:37:04+00:00" }, { "id": "0b46dfb4c4e14258bf88511f9099a03a", "sender": "jnewbery", "payload": "17167 has needed rebase since December", "action": false, "timestamp": "2021-03-04T19:37:26+00:00" }, { "id": "4877f470e560433f895c9d7d99ff355f", "sender": "MarcoFalke", "payload": "jnewbery: Can we replace fRelay with feefilter=max?", "action": false, "timestamp": "2021-03-04T19:37:27+00:00" }, { "id": "772a40efa3ee449abe7f564dfb13f686", "sender": "jonasschnelli", "payload": "The current wallet broadcast in normal operations isn't privacy preserving either (AFAIK without dandelion or similar)?", "action": false, "timestamp": "2021-03-04T19:37:33+00:00" }, { "id": "d074420521ef4863a9a44632207dfcd9", "sender": "jonatack", "payload": "FWIW regarding fRelay, I mentally rename it to m_wants_tx_relay", "action": false, "timestamp": "2021-03-04T19:37:36+00:00" }, { "id": "2b08e9288e344c15bb159ef29c5ecf1e", "sender": "luke-jr", "payload": "if there's renewed interest I can go ahead and rebase it", "action": false, "timestamp": "2021-03-04T19:37:40+00:00" }, { "id": "6833d5a1f7014e749370afdc64b8d0b6", "sender": "MarcoFalke", "payload": "jonasschnelli: I think the wallet broadcast has been reworked to be less agressive", "action": false, "timestamp": "2021-03-04T19:38:13+00:00" }, { "id": "f3208f3c83294b848fa94293670710c4", "sender": "sdaftuar", "payload": "MarcoFalke: would we then require that the peer support feefilter to stay connected?", "action": false, "timestamp": "2021-03-04T19:38:28+00:00" }, { "id": "ae8b2d145a194eb5bd252c86724f78db", "sender": "jonasschnelli", "payload": "sure. AFAIK chainanalysis companies are still able to capture it", "action": false, "timestamp": "2021-03-04T19:38:53+00:00" }, { "id": "57297520c98a4b86a9dea4c3b0ee9973", "sender": "amiti", "payload": "jonasschnelli: no its not, but I'm working on it :) #21061", "action": false, "timestamp": "2021-03-04T19:39:06+00:00" }, { "id": "e059af2f211f46eeae0086d8aab97d40", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/21061 | [p2p] Introduce node rebroadcast module by amitiuttarwar \u00c3\u0082\u00c2\u00b7 Pull Request #21061 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:39:08+00:00" }, { "id": "a133cc75fa3b405787026fd4cb31b5fa", "sender": "jonasschnelli", "payload": "amiti: nice", "action": false, "timestamp": "2021-03-04T19:39:17+00:00" }, { "id": "0d1cca0a61684e67a9d8f2a60b67df33", "sender": "jonasschnelli", "payload": "Just saying because if we flag blockonly privacy leaking we imply normal operation isn't", "action": false, "timestamp": "2021-03-04T19:39:42+00:00" }, { "id": "26676234a7524aeb93bdd6d391b11e1c", "sender": "MarcoFalke", "payload": "sdaftuar: We also assume the node to support BIP 60 for blocksonly, so it is not a large change?", "action": false, "timestamp": "2021-03-04T19:39:54+00:00" }, { "id": "93b945e7215e4e14af38dbde5aaa993d", "sender": "sipa", "payload": "if we believe that wallet privacy is a lost cause, there is no reason to improve it; but i don't think it is, and with improvements in the pipeline i don't think suboptimal privacy right now is a reason not to think about other issues", "action": false, "timestamp": "2021-03-04T19:40:11+00:00" }, { "id": "b1b447eaee5e4aa491fe2d7dc969337c", "sender": "MarcoFalke", "payload": "(nodes will be disconnected if they don't support BIP 60, no?)", "action": false, "timestamp": "2021-03-04T19:40:12+00:00" }, { "id": "09bd80381f054715822be4980b7d7874", "sender": "sdaftuar", "payload": "MarcoFalke: well my guess is that bip37/bip60 is more widely supported on the network. feefilter has always been optional", "action": false, "timestamp": "2021-03-04T19:40:18+00:00" }, { "id": "e720ebcec92a4944afff842fcde5c344", "sender": "jnewbery", "payload": "MarcoFalke: BIP60 is version 70002. feefilter is version 70013.", "action": false, "timestamp": "2021-03-04T19:40:20+00:00" }, { "id": "773810d24c854f98947210b39cc6b97d", "sender": "amiti", "payload": "sipa: strong +1", "action": false, "timestamp": "2021-03-04T19:40:28+00:00" }, { "id": "12270a105da34d439c7e2579331fbacb", "sender": "jonasschnelli", "payload": "yes. thats a good point sipa", "action": false, "timestamp": "2021-03-04T19:40:43+00:00" }, { "id": "0c3b6d7a4d9f4ed68788bf75fea53288", "sender": "jnewbery", "payload": "but I think being able to send feefilter before verack would be nice", "action": false, "timestamp": "2021-03-04T19:41:00+00:00" }, { "id": "016af683ec2d4858858d24a779374af0", "sender": "sipa", "payload": "still, i'm not sure exactly what we're talking about? blocksonly + sendrawtransaction? or also blocksonly + walletbroadcast + wallet send? something else?", "action": false, "timestamp": "2021-03-04T19:41:39+00:00" }, { "id": "6ab18be16b5f4daa9f85f3943d92ccac", "sender": "sdaftuar", "payload": "but wait feefilter doesn't help right jnewbery?", "action": false, "timestamp": "2021-03-04T19:41:46+00:00" }, { "id": "7152c3775bda424aa89ee91abbc6fe4b", "sender": "jnewbery", "payload": "feefilter doesn't help here, no", "action": false, "timestamp": "2021-03-04T19:42:02+00:00" }, { "id": "b00e5747fc694cf2a083997dadbbfb04", "sender": "jnewbery", "payload": "we'd still need BIP 338", "action": false, "timestamp": "2021-03-04T19:42:06+00:00" }, { "id": "086ade7bef594d2e8c2f573e0fae4fe5", "sender": "wumpus", "payload": "both security and privacy are always about making attacks marginally more difficult/expensive, not absolutes, would not call it a lost cause", "action": false, "timestamp": "2021-03-04T19:42:15+00:00" }, { "id": "3bf4b94ab0df44e4a64c96c21a44e892", "sender": "jnewbery", "payload": "some way to signal tx relay", "action": false, "timestamp": "2021-03-04T19:42:31+00:00" }, { "id": "7b8edd3b5f144cdc8dfb28567ea6725c", "sender": "sipa", "payload": "wumpus: right, my point was that sometimes an argument of \"you're trying to improve X, but X is always going to be bad because of Y, and we can't fix Y\" is a valid argument not to work on something... but i don't think that's the case here", "action": false, "timestamp": "2021-03-04T19:43:18+00:00" }, { "id": "3201cb11093b467abc569a3d1454f738", "sender": "jnewbery", "payload": "sipa: the specific reason I'm asking is that other than these edge cases, fRelay=false means \"I don't want txs on this connection and I'll never send txs on this connection\". That's a nice property", "action": false, "timestamp": "2021-03-04T19:44:07+00:00" }, { "id": "ae74853a9de646c0a6ce599b6ffbd3e8", "sender": "sipa", "payload": "what happens right now if someone runs bitcoin-qt in -blocksonly mode and creates a transaction?", "action": false, "timestamp": "2021-03-04T19:44:23+00:00" }, { "id": "06eb07a98b99437597f213326bc38945", "sender": "sdaftuar", "payload": "that's only true in a world where NODE_BLOOM isn't supported", "action": false, "timestamp": "2021-03-04T19:44:25+00:00" }, { "id": "930e61e280ec4045853b29c447e214d9", "sender": "jnewbery", "payload": "I was trying to understand if those edge cases are actually important, or whether they're just things that people are speculating about", "action": false, "timestamp": "2021-03-04T19:44:37+00:00" }, { "id": "ae8e7b75911d47958ab25debb0af14e2", "sender": "jonatack", "payload": "jnewbery: only the first of the two?", "action": false, "timestamp": "2021-03-04T19:44:39+00:00" }, { "id": "82c998e9079c4a7f8427a90a18c9edb6", "sender": "jnewbery", "payload": "*only true for nodes that don't support serving bloom filters, yes", "action": false, "timestamp": "2021-03-04T19:44:59+00:00" }, { "id": "195b0e3ea1aa474cbe5d3dd4022fc0d4", "sender": "luke-jr", "payload": "jnewbery: didn't sdaftuar just add a new flag for that?", "action": false, "timestamp": "2021-03-04T19:45:08+00:00" }, { "id": "3112d073384a478abff35e7c604d9470", "sender": "sipa", "payload": "while still supporting fRelay", "action": false, "timestamp": "2021-03-04T19:45:17+00:00" }, { "id": "a6ab5e3e20c54dac8ff6926bed9d9168", "sender": "wumpus", "payload": "sipa: sure, agree", "action": false, "timestamp": "2021-03-04T19:45:18+00:00" }, { "id": "6c7da7a4a25d4a299a31e8ec5c7fe6af", "sender": "sdaftuar", "payload": "luke-jr: not merged yet", "action": false, "timestamp": "2021-03-04T19:45:50+00:00" }, { "id": "f2ded787a907407d972806a3bba2e6f6", "sender": "jnewbery", "payload": "sipa: it wouldn't be broadcast because walletbroadcast is set to false. I don't know what the GUI shows", "action": false, "timestamp": "2021-03-04T19:45:52+00:00" }, { "id": "608bc97fa1824272b3d46fc96953dc79", "sender": "sipa", "payload": "that seems bad", "action": false, "timestamp": "2021-03-04T19:46:14+00:00" }, { "id": "af7b33254aef42e4a46e3e8e7bcf63bf", "sender": "luke-jr", "payload": "sdaftuar: right, but it would make fRelay having this property redundant..?", "action": false, "timestamp": "2021-03-04T19:46:24+00:00" }, { "id": "6846e0f024874e3a99337aa3a0b31847", "sender": "sipa", "payload": "conjectures that barely anyone uses -blocksonly on nodes they actually use as wallets", "action": true, "timestamp": "2021-03-04T19:46:33+00:00" }, { "id": "4b865ca2629a4159bbb0e0f8b2aec2f3", "sender": "jnewbery", "payload": "this was also my conjecture", "action": false, "timestamp": "2021-03-04T19:46:44+00:00" }, { "id": "e015c66177d04d24b6debb2883532f93", "sender": "sdaftuar", "payload": "luke-jr: i think jnewbery's proposal is to scrap the BIP", "action": false, "timestamp": "2021-03-04T19:46:47+00:00" }, { "id": "9136b926c09e461bb41f5cf89be8c762", "sender": "luke-jr", "payload": "ah", "action": false, "timestamp": "2021-03-04T19:46:51+00:00" }, { "id": "ea1b55f7d7264ce6a1c51510e98ecff7", "sender": "wumpus", "payload": "i don't think blocksonly was ever officially supported for the GUI, it works, sure", "action": false, "timestamp": "2021-03-04T19:47:28+00:00" }, { "id": "4a6a68c432d84972b9d2e9f0aa08b591", "sender": "MarcoFalke", "payload": "so replace BIP338 with a breaking p2p change and blocksonly meaning \"feefilter=max\"?", "action": false, "timestamp": "2021-03-04T19:47:30+00:00" }, { "id": "b73a97720e7c4eae86c0cf97d1b78c30", "sender": "jnewbery", "payload": "my original proposal was to script fRelay, but it seems that people rely on it to mean 'blocks only'", "action": false, "timestamp": "2021-03-04T19:47:42+00:00" }, { "id": "55bc1e047edc4c7c9c136d92a7379138", "sender": "wumpus", "payload": "but it was originally intended to be some kind of internal, rarely used thing", "action": false, "timestamp": "2021-03-04T19:47:42+00:00" }, { "id": "3ed1b03c701c4b36ba48af5082972458", "sender": "wumpus", "payload": "not something GUI end users would face", "action": false, "timestamp": "2021-03-04T19:47:52+00:00" }, { "id": "fdf2ddb94e6840fb9b7a9a140d6c585b", "sender": "jnewbery", "payload": "*scrap fRelay", "action": false, "timestamp": "2021-03-04T19:47:55+00:00" }, { "id": "1d9b51bd44df423f9bbf83b8b3cae1e7", "sender": "ariard", "payload": "fwiw, core doesn't even commit to bip60 as a supported bip", "action": false, "timestamp": "2021-03-04T19:48:09+00:00" }, { "id": "a9013a6adc8c44aea7f878b49016c886", "sender": "MarcoFalke", "payload": "ariard: It only does when -blocksonly", "action": false, "timestamp": "2021-03-04T19:48:41+00:00" }, { "id": "57572ba1bf804abca9eceb384b009c25", "sender": "darosior", "payload": "Fwiw i believe the GUI would show a fee estimation error in this case if no fallbackfee is set", "action": false, "timestamp": "2021-03-04T19:48:41+00:00" }, { "id": "849b12db0b8d4982931c9c93761d1412", "sender": "wumpus", "payload": "e.g. there is no configuration dialog option to enable it", "action": false, "timestamp": "2021-03-04T19:48:43+00:00" }, { "id": "b5c92d4660e342b19e066ae7889c0d31", "sender": "sipa", "payload": "darosior: fair point", "action": false, "timestamp": "2021-03-04T19:48:54+00:00" }, { "id": "6d4a4350e26a4be3a595c1c992e03a73", "sender": "ariard", "payload": "MarcoFalke: i mean here https://github.com/bitcoin/bitcoin/blob/master/doc/bips.md, for a dev writing another bitcoin client and trying to understand core behavior", "action": false, "timestamp": "2021-03-04T19:49:14+00:00" }, { "id": "f2dc9bfc66594cf29ed0b53625d53711", "sender": "wumpus", "payload": "darosior: i think so too", "action": false, "timestamp": "2021-03-04T19:49:22+00:00" }, { "id": "038f3398c74241c0b2dec6a2522e240e", "sender": "sipa", "payload": "we don't support bip60 and never have", "action": false, "timestamp": "2021-03-04T19:49:27+00:00" }, { "id": "7aae21f835604f51a50cb3d2160d226f", "sender": "jonasschnelli", "payload": "darosior: Yes. It will. But users with just set a manual fee", "action": false, "timestamp": "2021-03-04T19:49:34+00:00" }, { "id": "678a9607cc3d487eb7fc0a749ece402b", "sender": "sipa", "payload": "or at least, we don't assume our peers do", "action": false, "timestamp": "2021-03-04T19:49:38+00:00" }, { "id": "b10744ac67bb4253a1a302b950376fc4", "sender": "luke-jr", "payload": "sipa: we send it", "action": false, "timestamp": "2021-03-04T19:49:44+00:00" }, { "id": "0e44b64f86e5452b85e093faeaf26796", "sender": "luke-jr", "payload": "iirc", "action": false, "timestamp": "2021-03-04T19:49:55+00:00" }, { "id": "0e40c5af00db46c096e0d645fc31f08c", "sender": "jnewbery", "payload": "we always send fRelay, but don't expect our peers to", "action": false, "timestamp": "2021-03-04T19:49:56+00:00" }, { "id": "fd0b5b7f1ec64a538266232614835c55", "sender": "sipa", "payload": "luke-jr: BIP60 is about predictably-sized version messages, not just the fRelay flag", "action": false, "timestamp": "2021-03-04T19:50:15+00:00" }, { "id": "326aabe1d4cb484fa8d5cb83c5b41afb", "sender": "jnewbery", "payload": "we also don't require that our peers send starting height, subversion, etc", "action": false, "timestamp": "2021-03-04T19:50:57+00:00" }, { "id": "2eb0f45d7a8449ba84a8b62191cd75ca", "sender": "luke-jr", "payload": "sipa: but on the sender side", "action": false, "timestamp": "2021-03-04T19:50:58+00:00" }, { "id": "c3707df3e0a14ef8b66e38833cb4c8fd", "sender": "sdaftuar", "payload": "sipa: but we're compatible with bip60 peers, and i think we now expect our peers to always read our fRelay flag if its' set to false, right?", "action": false, "timestamp": "2021-03-04T19:51:00+00:00" }, { "id": "ff1aefed8154491bb9a797f33c57e5e6", "sender": "ariard", "payload": "even the original bip37 is really unclear if fRelay semantic is to disable tx-relay or just tx-announcement", "action": false, "timestamp": "2021-03-04T19:51:02+00:00" }, { "id": "5e74762bb47d490ab69fa89b08c009a8", "sender": "luke-jr", "payload": "it doesn't require breakign non-BIP60 peers", "action": false, "timestamp": "2021-03-04T19:51:05+00:00" }, { "id": "c2501f066a8c4903bd499c90d6f75c07", "sender": "sipa", "payload": "sdaftuar: hmm, true", "action": false, "timestamp": "2021-03-04T19:51:10+00:00" }, { "id": "fc092a42da4949428906b4a2bbbc1ea4", "sender": "jnewbery", "payload": "sdaftuar: you mean we expect them to read fRelay because we'll disconnect them if they send us a tx?", "action": false, "timestamp": "2021-03-04T19:52:20+00:00" }, { "id": "0cd633a87b854d2bb8478336c54e8513", "sender": "sdaftuar", "payload": "right, if we set fRelay=false i think we now disconnect if they ignore it", "action": false, "timestamp": "2021-03-04T19:52:34+00:00" }, { "id": "24278a5f14e84e52aae2bf7448901f5a", "sender": "ariard", "payload": "unless you signal NODE_BLOOM", "action": false, "timestamp": "2021-03-04T19:52:42+00:00" }, { "id": "2af01716d51f40e0b0ba938a987125a6", "sender": "MarcoFalke", "payload": "Bitcoin Core implements BIP60, but doesn't require peers to implement it. (Like any other optional BIP)", "action": false, "timestamp": "2021-03-04T19:53:16+00:00" }, { "id": "78ec1ddc7e0b49f28b69245aff0f392d", "sender": "jnewbery", "payload": "I'm not sure we do disconnect peers for that", "action": false, "timestamp": "2021-03-04T19:53:38+00:00" }, { "id": "3fa2af55849c4a22a4c3db411c8d6955", "sender": "amiti", "payload": "orly? I thought we disconnected for that too", "action": false, "timestamp": "2021-03-04T19:53:58+00:00" }, { "id": "a907e49f845c4c0598bd98cf687b8317", "sender": "sdaftuar", "payload": "jnewbery: it is possible i'm misremembering , i know i thought this at some point in the past and was wrong", "action": false, "timestamp": "2021-03-04T19:54:02+00:00" }, { "id": "af4b4c099f764ff2b36313d0e8b8d4f6", "sender": "sipa", "payload": "so i guess we could list BIP60 as implemented, actually", "action": false, "timestamp": "2021-03-04T19:54:09+00:00" }, { "id": "bbf6c4ed12454ab7b80ab4ee335a43f1", "sender": "MarcoFalke", "payload": "jnewbery: I am pretty sure there is a \"tx sent in violation of protocol\" log", "action": false, "timestamp": "2021-03-04T19:54:21+00:00" }, { "id": "93694248761240fd8daf219019ba7641", "sender": "ariard", "payload": "iirc #15759 introduce the change of disconnecting block-relay-only peers sending us a transaction", "action": false, "timestamp": "2021-03-04T19:54:26+00:00" }, { "id": "59cbc65c8117430d8ed1e14238b31944", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/15759 | p2p: Add 2 outbound block-relay-only connections by sdaftuar \u00c3\u0082\u00c2\u00b7 Pull Request #15759 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:54:30+00:00" }, { "id": "272909da728441cf8d946bcbf6b5bd45", "sender": "jnewbery", "payload": "but not if we send fRelay=false because we're in -blocksonly", "action": false, "timestamp": "2021-03-04T19:55:08+00:00" }, { "id": "485f06aae4d64fa1baebfeffbcdf376b", "sender": "wumpus", "payload": "5 minutes to go for the meeting, ajonas still had an announcement wrt the developer survey", "action": false, "timestamp": "2021-03-04T19:55:24+00:00" }, { "id": "b54842e965784baf89682cacfb9505e9", "sender": "MarcoFalke", "payload": "jnewbery: Oh, I didn't know", "action": false, "timestamp": "2021-03-04T19:55:28+00:00" }, { "id": "af680a9576d346e2ba5b02af3c6abb7f", "sender": "jnewbery", "payload": "I may be wrong. I searched for fRelayTxes in net_processing", "action": false, "timestamp": "2021-03-04T19:55:47+00:00" }, { "id": "48127eb2e2604d478fa224fd3a44cdaa", "sender": "sdaftuar", "payload": "looks likew e disconnect in that case to me? can look later jnewbery", "action": false, "timestamp": "2021-03-04T19:55:50+00:00" }, { "id": "e3cbe9413bc34eecb2f28a61155fb938", "sender": "amiti", "payload": "https://github.com/bitcoin/bitcoin/blob/master/src/net_processing.cpp#L3001", "action": false, "timestamp": "2021-03-04T19:55:50+00:00" }, { "id": "ec08308c599449248aeee6a0aceda20a", "sender": "MarcoFalke", "payload": "(Side note: we also don't disconnect on feefilter=max violations)", "action": false, "timestamp": "2021-03-04T19:56:07+00:00" }, { "id": "6fafad94c4184f68b1e433a7a4cd8ffd", "sender": "ariard", "payload": "jnewbery: correct this was a pre-15759 behavior", "action": false, "timestamp": "2021-03-04T19:56:13+00:00" }, { "id": "bd47dd535bbe4924a99bfcfede63a499", "sender": "jnewbery", "payload": "sdaftuar: ah, you're right. Thanks!", "action": false, "timestamp": "2021-03-04T19:56:26+00:00" }, { "id": "d8f55b4e160a4352bb734b5434727816", "sender": "jnewbery", "payload": "ok, shall we let jonas have his three minutes? :)", "action": false, "timestamp": "2021-03-04T19:56:39+00:00" }, { "id": "7d15e734411545228e801c95b281cc20", "sender": "phantomcircuit", "payload": "sipa, would the headers sync thing be appropriate in this meeting?", "action": false, "timestamp": "2021-03-04T19:56:41+00:00" }, { "id": "3aca41b5701747a4af81744de4dae9ea", "sender": "wumpus", "payload": "#topic Developer survey (ajonas)", "action": false, "timestamp": "2021-03-04T19:56:42+00:00" }, { "id": "1b0ad9ef45d54792973028f91e5e83ca", "sender": "MarcoFalke", "payload": "m_ignore_incoming_txs means \"blocksonly\"", "action": false, "timestamp": "2021-03-04T19:56:54+00:00" }, { "id": "bd31d3cf1f394cf68425eaebd13f3a30", "sender": "ajonas", "payload": "Hi. A summary of the latest dev survey is at https://adamjonas.com/bitcoin/coredev/retro/coredev-2020-retro/. Thank you to everyone who participated.", "action": false, "timestamp": "2021-03-04T19:56:55+00:00" }, { "id": "0729682c8b9344ffa950b745f79a8c5d", "sender": "MarcoFalke", "payload": "ajonas: Thanks for summing those up!", "action": false, "timestamp": "2021-03-04T19:57:06+00:00" }, { "id": "aeef8941b64740dabd89f0fb7ab5f6e9", "sender": "jnewbery", "payload": "MarcoFalke: yup. You're right", "action": false, "timestamp": "2021-03-04T19:57:07+00:00" }, { "id": "87ae45d9648647bd9c4afbde364d96bb", "sender": "wumpus", "payload": "ajonas: thanks for doing this", "action": false, "timestamp": "2021-03-04T19:57:13+00:00" }, { "id": "5c6253bb8f23450e80c1bfa26217fb27", "sender": "ajonas", "payload": "Happy to!", "action": false, "timestamp": "2021-03-04T19:57:27+00:00" }, { "id": "183bdf8b35cd4bb8aae1f9cbf8d9c40a", "sender": "jnewbery", "payload": "thank you ajonas!", "action": false, "timestamp": "2021-03-04T19:57:42+00:00" }, { "id": "d283907c90fa48d3ad0adbbb0647bfa0", "sender": "MarcoFalke", "payload": "One major theme is \"It is unclear which status a pr is in\". I was thinking how to improve that, but couldn't find a solution", "action": false, "timestamp": "2021-03-04T19:57:52+00:00" }, { "id": "921ef4c3d32a4c9ea05c740c9616eeb6", "sender": "ajonas", "payload": "I think a labeling system might be worth trialing", "action": false, "timestamp": "2021-03-04T19:58:15+00:00" }, { "id": "c1a2e39d94a04d45a5d6cc9dd8c18763", "sender": "MarcoFalke", "payload": "Obviously it can't be an objective metric (that a computer could measure), because then it would be gameable", "action": false, "timestamp": "2021-03-04T19:58:24+00:00" }, { "id": "8228789b6536403eb0a585213eb7b1b8", "sender": "amiti", "payload": "imo, communicating PR status should be the responsibility of the pr author instead of maintainers", "action": false, "timestamp": "2021-03-04T19:58:32+00:00" }, { "id": "a3a00779afcd4275906fb5a078e0eb54", "sender": "ajonas", "payload": "But we could also add some meta data in the first comment that can be updated over time", "action": false, "timestamp": "2021-03-04T19:58:52+00:00" }, { "id": "38ddff19ef4f4e13b2e47cfb67a061ff", "sender": "wumpus", "payload": "what are the PR statuses in the first place?", "action": false, "timestamp": "2021-03-04T19:58:59+00:00" }, { "id": "f957efa86b90460fa6eddfcd2934bc04", "sender": "MarcoFalke", "payload": "I think the best metric is a heuristic \"A pr is closer to merge the more ACKs it has relative to its potential risk\"", "action": false, "timestamp": "2021-03-04T19:59:08+00:00" }, { "id": "f6327e2a17124f829f11776df0b216b4", "sender": "wumpus", "payload": "i mean we have a few like 'waiting for author' 'needs rebase' but not sure what you mean here", "action": false, "timestamp": "2021-03-04T19:59:22+00:00" }, { "id": "2f19cc37490b42159a537031be70a80f", "sender": "wumpus", "payload": "MarcoFalke: yes, definitely", "action": false, "timestamp": "2021-03-04T19:59:43+00:00" }, { "id": "b41b8f20ca1c471abe63e4ad37895ca9", "sender": "ajonas", "payload": "I think concept / approach ack vs deep code review", "action": false, "timestamp": "2021-03-04T19:59:43+00:00" }, { "id": "694d9db149bc47f9b9fa96222f07fcda", "sender": "MarcoFalke", "payload": "wumpus: I'd say when there is no \"needs rebase\" label, the status is implicitly \"needs review\"", "action": false, "timestamp": "2021-03-04T19:59:53+00:00" }, { "id": "6ebf8584b681470eb58f13ab6353cd41", "sender": "wumpus", "payload": "ok so like 'needs concept ACK'", "action": false, "timestamp": "2021-03-04T20:00:04+00:00" }, { "id": "4fc230537c074b48b66967588488064c", "sender": "wumpus", "payload": "we could add that, but yes, like amiti says, authors have to request this", "action": false, "timestamp": "2021-03-04T20:00:21+00:00" }, { "id": "732d939dff19422d9e881a0edf460d67", "sender": "ariard", "payload": "draft status doesn't mean \"needs concept ACK\"?", "action": false, "timestamp": "2021-03-04T20:00:46+00:00" }, { "id": "a9e29bd723c04c00a295d1bcbdf9826a", "sender": "MarcoFalke", "payload": "I am not sure if that makes sense. How would you know when to remove the \"needs concept ACK\" label?", "action": false, "timestamp": "2021-03-04T20:00:55+00:00" }, { "id": "b76f6e1a88a24984bfccdf6d659526ba", "sender": "wumpus", "payload": "ariard: maybe", "action": false, "timestamp": "2021-03-04T20:00:59+00:00" }, { "id": "c0f20060962a457e89600aa6b66bf4c1", "sender": "ajonas", "payload": "I think the use of draft is inconsistent", "action": false, "timestamp": "2021-03-04T20:01:05+00:00" }, { "id": "abbf5e930e3940179bb60a91eb88d989", "sender": "wumpus", "payload": "ariard: we could use it for that", "action": false, "timestamp": "2021-03-04T20:01:08+00:00" }, { "id": "9984770fbe5d4e328028b8ac63d47fe3", "sender": "wumpus", "payload": "MarcoFalke: good point", "action": false, "timestamp": "2021-03-04T20:01:21+00:00" }, { "id": "01c18ebd1e4c402fb1e88806f8e23375", "sender": "ariard", "payload": "good thing with draft you can do it just a pr author, doesn't special repo permisions iirc", "action": false, "timestamp": "2021-03-04T20:01:28+00:00" }, { "id": "c6f622a84e4c49feb73c7369d3e0a0fe", "sender": "wumpus", "payload": "yes that is good", "action": false, "timestamp": "2021-03-04T20:01:42+00:00" }, { "id": "3e64a4d177774fd189e371d65e84b33a", "sender": "wumpus", "payload": "self-organizaing is good, anything that gives the maintainers more on their plate is probably not going to fly", "action": false, "timestamp": "2021-03-04T20:02:17+00:00" }, { "id": "165d498e9c3d46f297e557dbca5fbcae", "sender": "amiti", "payload": "something I'm trying on #21061 is adding an explicit \"status\" section at the top of OP. not super relevant yet because the PR hasn't gotten much attention, but was hoping to make this easier for reviewers as the comments accumulate", "action": false, "timestamp": "2021-03-04T20:03:15+00:00" }, { "id": "d99b90ef87a7441290580041267ce73f", "sender": "wumpus", "payload": "that concludes the meeting for today", "action": false, "timestamp": "2021-03-04T20:03:15+00:00" }, { "id": "0f71cccbb54b425785d5016be2e88035", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/21061 | [p2p] Introduce node rebroadcast module by amitiuttarwar \u00c3\u0082\u00c2\u00b7 Pull Request #21061 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T20:03:17+00:00" }, { "id": "056af037d74945a3b27da2a7bd320421", "sender": "wumpus", "payload": "#endmeeting", "action": false, "timestamp": "2021-03-04T20:03:27+00:00" } ], "events": [ { "event_type": "START_MEETING", "message": { "id": "d0ddc473209245608995fd0b29edb858", "sender": "wumpus", "payload": "#startmeeting", "action": false, "timestamp": "2021-03-04T19:00:55+00:00" }, "operand": null, "id": "d0ddc473209245608995fd0b29edb858", "timestamp": "2021-03-04T19:00:55+00:00" }, { "event_type": "TOPIC", "message": { "id": "81bfe455014847a4869a7a5d38baf778", "sender": "wumpus", "payload": "#topic High priority for review", "action": false, "timestamp": "2021-03-04T19:03:43+00:00" }, "operand": "High priority for review", "id": "81bfe455014847a4869a7a5d38baf778", "timestamp": "2021-03-04T19:03:43+00:00" }, { "event_type": "LINK", "message": { "id": "63b41d6b6e8f4fdb8839cc3a8a252e28", "sender": "wumpus", "payload": "https://github.com/bitcoin/bitcoin/projects/8 9 blockers, 2 chasing concept ACK", "action": false, "timestamp": "2021-03-04T19:04:00+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/projects/8", "id": "63b41d6b6e8f4fdb8839cc3a8a252e28", "timestamp": "2021-03-04T19:04:00+00:00" }, { "event_type": "LINK", "message": { "id": "90ee9703efd9454581b1dd480452ea8c", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/20664 | Add scanblocks RPC call by jonasschnelli \u00c3\u0082\u00c2\u00b7 Pull Request #20664 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:04:00+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/20664", "id": "90ee9703efd9454581b1dd480452ea8c", "timestamp": "2021-03-04T19:04:00+00:00" }, { "event_type": "LINK", "message": { "id": "fcd609de7aa44522a035a2aa6816e1d8", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/20197 | p2p: protect onions in AttemptToEvictConnection(), add eviction protection test coverage by jonatack \u00c3\u0082\u00c2\u00b7 Pull Request #20197 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:04:50+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/20197", "id": "fcd609de7aa44522a035a2aa6816e1d8", "timestamp": "2021-03-04T19:04:50+00:00" }, { "event_type": "LINK", "message": { "id": "2b51ec0687eb4fc0bc70078d630dde85", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/20536 | wallet: Error with \"Transaction too large\" if the funded tx will end up being too large after signing by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #20536 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:07:53+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/20536", "id": "2b51ec0687eb4fc0bc70078d630dde85", "timestamp": "2021-03-04T19:07:53+00:00" }, { "event_type": "LINK", "message": { "id": "bead1dc524714e3ebd90129b1224b4fe", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/18795 | Test: wallet issue with orphaned rewards by domob1812 \u00c3\u0082\u00c2\u00b7 Pull Request #18795 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:08:55+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/18795", "id": "bead1dc524714e3ebd90129b1224b4fe", "timestamp": "2021-03-04T19:08:55+00:00" }, { "event_type": "LINK", "message": { "id": "e581e4b6089f4fdebf77fdcab3514fd1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/19393 | test: Add more tests for orphan tx handling by hebasto \u00c3\u0082\u00c2\u00b7 Pull Request #19393 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:08:57+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/19393", "id": "e581e4b6089f4fdebf77fdcab3514fd1", "timestamp": "2021-03-04T19:08:57+00:00" }, { "event_type": "LINK", "message": { "id": "abb86a8799f5453199f55e81cae49f61", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/14604 | tests: Add test and refactor feature_block.py by sanket1729 \u00c3\u0082\u00c2\u00b7 Pull Request #14604 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:08:59+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/14604", "id": "abb86a8799f5453199f55e81cae49f61", "timestamp": "2021-03-04T19:08:59+00:00" }, { "event_type": "TOPIC", "message": { "id": "59bc62a531c84c4fa0adcb249a06f4dc", "sender": "wumpus", "payload": "#topic Blocksonly mode and transaction relay (jnewbery)", "action": false, "timestamp": "2021-03-04T19:09:39+00:00" }, "operand": "Blocksonly mode and transaction relay (jnewbery)", "id": "59bc62a531c84c4fa0adcb249a06f4dc", "timestamp": "2021-03-04T19:09:39+00:00" }, { "event_type": "LINK", "message": { "id": "56eff8f0f7104a76a4c773ac71134a63", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/6993 | Add -blocksonly option by pstratem \u00c3\u0082\u00c2\u00b7 Pull Request #6993 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:10:26+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/6993", "id": "56eff8f0f7104a76a4c773ac71134a63", "timestamp": "2021-03-04T19:10:26+00:00" }, { "event_type": "LINK", "message": { "id": "4b6cb3127bb043dba46e3ee042e57dd1", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/15990 | Add tests and documentation for blocksonly by MarcoFalke \u00c3\u0082\u00c2\u00b7 Pull Request #15990 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:10:29+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/15990", "id": "4b6cb3127bb043dba46e3ee042e57dd1", "timestamp": "2021-03-04T19:10:29+00:00" }, { "event_type": "LINK", "message": { "id": "f6411ae15b3c4ad88f6dc824cb9c3c6d", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/4 | Export/Import wallet in a human readable, future-proof format \u00c3\u0082\u00c2\u00b7 Issue #4 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:10:31+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/4", "id": "f6411ae15b3c4ad88f6dc824cb9c3c6d", "timestamp": "2021-03-04T19:10:31+00:00" }, { "event_type": "LINK", "message": { "id": "648a1f9b5fec4293b0e94e6ec6becd56", "sender": "MarcoFalke", "payload": "https://github.com/bitcoin/bitcoin/blob/master/doc/reduce-traffic.md#4-turn-off-transaction-relay--blocksonly", "action": false, "timestamp": "2021-03-04T19:18:25+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/blob/master/doc/reduce-traffic.md#4-turn-off-transaction-relay--blocksonly", "id": "648a1f9b5fec4293b0e94e6ec6becd56", "timestamp": "2021-03-04T19:18:25+00:00" }, { "event_type": "LINK", "message": { "id": "38d38cfb12ba40668db89307fa602654", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/4 | Export/Import wallet in a human readable, future-proof format \u00c3\u0082\u00c2\u00b7 Issue #4 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:18:27+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/4", "id": "38d38cfb12ba40668db89307fa602654", "timestamp": "2021-03-04T19:18:27+00:00" }, { "event_type": "LINK", "message": { "id": "8fdfa2ebe0854395bf0fd433efd3e3c4", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/17167 | Allow whitelisting outgoing connections by luke-jr \u00c3\u0082\u00c2\u00b7 Pull Request #17167 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:36:38+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/17167", "id": "8fdfa2ebe0854395bf0fd433efd3e3c4", "timestamp": "2021-03-04T19:36:38+00:00" }, { "event_type": "LINK", "message": { "id": "e059af2f211f46eeae0086d8aab97d40", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/21061 | [p2p] Introduce node rebroadcast module by amitiuttarwar \u00c3\u0082\u00c2\u00b7 Pull Request #21061 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:39:08+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/21061", "id": "e059af2f211f46eeae0086d8aab97d40", "timestamp": "2021-03-04T19:39:08+00:00" }, { "event_type": "LINK", "message": { "id": "59cbc65c8117430d8ed1e14238b31944", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/15759 | p2p: Add 2 outbound block-relay-only connections by sdaftuar \u00c3\u0082\u00c2\u00b7 Pull Request #15759 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T19:54:30+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/15759", "id": "59cbc65c8117430d8ed1e14238b31944", "timestamp": "2021-03-04T19:54:30+00:00" }, { "event_type": "LINK", "message": { "id": "e3cbe9413bc34eecb2f28a61155fb938", "sender": "amiti", "payload": "https://github.com/bitcoin/bitcoin/blob/master/src/net_processing.cpp#L3001", "action": false, "timestamp": "2021-03-04T19:55:50+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/blob/master/src/net_processing.cpp#L3001", "id": "e3cbe9413bc34eecb2f28a61155fb938", "timestamp": "2021-03-04T19:55:50+00:00" }, { "event_type": "TOPIC", "message": { "id": "3aca41b5701747a4af81744de4dae9ea", "sender": "wumpus", "payload": "#topic Developer survey (ajonas)", "action": false, "timestamp": "2021-03-04T19:56:42+00:00" }, "operand": "Developer survey (ajonas)", "id": "3aca41b5701747a4af81744de4dae9ea", "timestamp": "2021-03-04T19:56:42+00:00" }, { "event_type": "LINK", "message": { "id": "0f71cccbb54b425785d5016be2e88035", "sender": "gribble", "payload": "https://github.com/bitcoin/bitcoin/issues/21061 | [p2p] Introduce node rebroadcast module by amitiuttarwar \u00c3\u0082\u00c2\u00b7 Pull Request #21061 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub", "action": false, "timestamp": "2021-03-04T20:03:17+00:00" }, "operand": "https://github.com/bitcoin/bitcoin/issues/21061", "id": "0f71cccbb54b425785d5016be2e88035", "timestamp": "2021-03-04T20:03:17+00:00" }, { "event_type": "END_MEETING", "message": { "id": "056af037d74945a3b27da2a7bd320421", "sender": "wumpus", "payload": "#endmeeting", "action": false, "timestamp": "2021-03-04T20:03:27+00:00" }, "operand": null, "id": "056af037d74945a3b27da2a7bd320421", "timestamp": "2021-03-04T20:03:27+00:00" } ], "aliases": {}, "vote_in_progress": false, "motion_index": null }