1 2017-02-13 00:09:18 0|bitcoin-git|[13bitcoin] 15luke-jr opened pull request #9749: If -spkreuse=0, ensure transactions in mempool always have unique scriptPubKeys (06master...06unique_spk_mempool) 02https://github.com/bitcoin/bitcoin/pull/9749
2 2017-02-13 00:46:16 0|gmaxwell|luke-jr: it would be interesting if there were a 1 bit flag in scriptpubkeys that indicated if you wanted to allow reuse or not.
3 2017-02-13 01:29:19 0|luke-jr|gmaxwell: enforcing it consistently would require an ever-growing set, so this is more useful to simply discourage it in general
4 2017-02-13 01:29:49 0|gmaxwell|it would just be "best effort"
5 2017-02-13 01:30:14 0|gmaxwell|e.g. I promise no two in the last N blocks, beyond that, "probablistic"
6 2017-02-13 01:30:24 0|luke-jr|I can't think of any reason to ever intentionally accomidate reuse.
7 2017-02-13 01:30:41 0|gmaxwell|widespread practice.
8 2017-02-13 01:31:03 0|gmaxwell|an improvement no one uses is not an improvement.
9 2017-02-13 01:31:25 0|luke-jr|widespread practice needs to change; people enabling this puts pressure on others to stop doing it
10 2017-02-13 01:31:47 0|luke-jr|especially people doing more transaction volume
11 2017-02-13 01:31:53 0|BlueMatt|how? only realistically if its off-by-default, which we cant do, or if miners use it, which they wont
12 2017-02-13 01:32:08 0|sdaftuar|i haven't read this patch carefully, but is there anything to prevent an attacker from interfering with relay by sending a low-fee, low-value transaction to an output that he sees, and relaying that ahead of the original tx?
13 2017-02-13 01:32:13 0|gmaxwell|and what if the net effect is just someone standarziding that wallets should notice their scriptpubkey with a dummy push also strapped to it?
14 2017-02-13 01:32:25 0|gmaxwell|oh slick point.
15 2017-02-13 01:32:33 0|luke-jr|sdaftuar: there isn't. perhaps it should use RBF semantics in that case
16 2017-02-13 01:33:03 0|sdaftuar|this seems like a solution looking for a problem imo
17 2017-02-13 01:33:20 0|gmaxwell|well there are problems it would help with.
18 2017-02-13 01:34:11 0|gmaxwell|for example,, if you take some old addresses and spend all the connected coins, people have frequently then sent near dust amounts to them, seemingly with the hope that you'll spend the dust in another transaction and link the outputs.
19 2017-02-13 01:34:59 0|morcos|luke-jr: also to clarify (sorry for not just reading the code), if there are already multiple outputs encumbered by the same scriptpubkey, does this patch prevent spending more than one of them at a time?
20 2017-02-13 01:35:19 0|gmaxwell|no, it's just a creation restriction AFAICT.
21 2017-02-13 01:35:20 0|luke-jr|morcos: it doesn't prevent that.
22 2017-02-13 01:35:34 0|luke-jr|it restricts spending as well, but allows multiple of them in the same tx
23 2017-02-13 01:35:35 0|morcos|ok, the exception in your PR text confused me.. that's good at least!
24 2017-02-13 01:35:41 0|luke-jr|brb
25 2017-02-13 01:35:53 0|morcos|wait... so it does prevent that unless you put them in the same TX????
26 2017-02-13 01:37:09 0|jcorgan|is there a particular reason bitcoin.conf only allows IP parameters by address and not hostname/dns name, other than "it hasn't been written yet"?
27 2017-02-13 01:37:50 0|morcos|for the record, i agree with the basic gist of discouraging address reuse, but i think its crazy to make it an absolute. i've reused (And continue to reuse) addresess many times... its a choice of convenience over.. eh.. maybe some loss of privacy
28 2017-02-13 01:38:32 0|morcos|for instance if your change output is readily discernible already..
29 2017-02-13 01:38:35 0|gmaxwell|luke-jr: probably more important is that more outputs shouldn't be made to an address once it's been spent from (and ideally wallets would spend from that address all at once)
30 2017-02-13 01:40:22 0|midnightmagic|morcos: Also, loss of privacy for other people who deal with you.
31 2017-02-13 02:26:16 0|luke-jr|morcos: it's not absolute; you can wait until the transaction is mined
32 2017-02-13 02:27:32 0|luke-jr|it occurs to me the current PR already breaks RBF, so I need to fix that :/
33 2017-02-13 02:31:18 0|luke-jr|I suppose it should just treat SPK overlaps as if they shared a conflicting input
34 2017-02-13 02:31:46 0|luke-jr|solve both issues at once
35 2017-02-13 02:32:36 0|morcos|i think it makes no sense at all to have an option that makes it difficult for people to spend previously created outputs
36 2017-02-13 02:32:47 0|morcos|i'd be opposed to that
37 2017-02-13 02:33:18 0|morcos|i mean i'm basically opposed to the option in general.. but if its just an option and it defaults off.. well you have to pick your battles
38 2017-02-13 02:33:27 0|morcos|but if it prevents spending existing outputs.. that seems crazy
39 2017-02-13 02:33:54 0|luke-jr|you can spend them in separate blocks or in the same tx
40 2017-02-13 02:34:04 0|luke-jr|there is no reason anyone should ever have multiple tx in the same block anyway
41 2017-02-13 02:35:25 0|sipa|right, but why would miners enable that setting?
42 2017-02-13 02:35:37 0|sipa|and if miners don't, why would anyone else?
43 2017-02-13 02:36:03 0|sipa|i like the reasons... but it's noneconomical
44 2017-02-13 02:36:08 0|luke-jr|because some miners actually care about Bitcoin
45 2017-02-13 02:36:26 0|morcos|honestly i think this is the kind of change that should go into something like Knots
46 2017-02-13 02:38:18 0|luke-jr|I don't like the direction Core is changing to where everything must be exclusively economic incentives. Bitcoin can't work with mere economic incentives as things are today. The way things are heading, Core is no longer a reference implementation, but a specific political agenda to the exclusion of others.
47 2017-02-13 02:39:16 0|sipa|luke-jr: the only criteria for non-miner mempools is the expectancy of what will confirm
48 2017-02-13 02:39:40 0|sipa|for miners, i would argue that any non-economical property is a push for a certain policy
49 2017-02-13 02:39:51 0|sipa|the economical one is that i expect everyone to take eventually
50 2017-02-13 02:40:11 0|luke-jr|sipa: I see it more of that non-miner mempools constrain miners, by creating slower propagation of blocks that don't match the wider network policies.
51 2017-02-13 02:40:23 0|luke-jr|sipa: economical is also a certain policy. options are options.
52 2017-02-13 02:40:52 0|sipa|luke-jr: divergence between non-miners and miners just encourages people to submit directly to miners, and miners to be reachable (and thus non-anonymous)
53 2017-02-13 02:41:23 0|sipa|i think that's a far worse outcome than just rational policies
54 2017-02-13 02:41:27 0|luke-jr|lack of divergence creates centralisation pressures forcing miners to all run the same policy dictated by developers
55 2017-02-13 02:41:38 0|morcos|i would argue that a minimum bar for an option is that at least the majority of Core developers think its a good idea OR a good fraction of the user community think its a good idea
56 2017-02-13 02:41:44 0|morcos|we can't support every fringe option
57 2017-02-13 02:42:47 0|sipa|luke-jr: my assumption is that if we'd introduce non-economical policies, and they're configurable they'd be turned off, and if they're not configurable someone will create a patch to change them
58 2017-02-13 02:43:07 0|sipa|by the vast majority of miners
59 2017-02-13 02:43:41 0|morcos|i also dislike the use of policy as something that tries to constrain users to use bitcoin in a certain way.. if its not necessary for DoS prevention or resource allocation, then good usage policies/options should be at the wallet level
60 2017-02-13 02:44:10 0|luke-jr|I don't see many miners turning on acceptnonstdtxn. Or enabling full RBF (although some do exist). Miners do care to an extent, and that will hopefully improve as difficulty kills profits.
61 2017-02-13 02:44:10 0|morcos|relay and mining should be essentially blind to any attribute of txs other than resource usage (which is unfortunately a multi-dimensional beast)
62 2017-02-13 02:45:45 0|luke-jr|morcos: that is not a sustainable view, at least as things are today. but more importantly, I concede your right to take that position, but you need to understand not everyone agrees with it.
63 2017-02-13 02:46:15 0|sipa|luke-jr: ok, i'll reformulate: my expectation is that over time we'll converge to more rational relay policies... not because developers force it, but because it's the most economical thing to do
64 2017-02-13 02:46:39 0|luke-jr|economical != rational
65 2017-02-13 02:46:52 0|sipa|short-term vs long-term?
66 2017-02-13 02:47:00 0|luke-jr|perhaps
67 2017-02-13 02:47:09 0|luke-jr|it's rational to filter SPK reuse in hopes of improving Bitcoin privacy, for example
68 2017-02-13 02:47:54 0|sipa|the only thing that actually improves privacy IMHO is consensus rules that incentivize it
69 2017-02-13 02:48:17 0|morcos|luke-jr: i'd be much more amenable to that argument if the Core wallet stopped SPK reuse.. having relay be difficult in the even of SPK reuse seems like it risks causing more harm than good
70 2017-02-13 02:48:29 0|luke-jr|neither developers nor miners control consensus rules. but miners do control policy.
71 2017-02-13 02:49:02 0|morcos|luke-jr: right! that's why we need to work towards a bitcoin where there is no policy!! all txs look the same
72 2017-02-13 02:49:17 0|luke-jr|no, policy is important.
73 2017-02-13 02:49:18 0|sipa|yes, let's switch to MimbleWimble. all txs look the same!
74 2017-02-13 02:49:56 0|sipa|luke-jr: in my ideal world, there can't be any policy beyond fees/size, because there is nothing else that distinguishes two transactions
75 2017-02-13 02:50:16 0|luke-jr|sipa: that's not everyone's ideal world.
76 2017-02-13 02:50:30 0|sipa|the fact that you can even say whether two outputs use the same address is a fungibility flaw on itself
77 2017-02-13 02:50:46 0|sipa|the fact that miners have censorship rights at all is a weakness
78 2017-02-13 02:50:48 0|luke-jr|I suppose
79 2017-02-13 02:50:54 0|luke-jr|no, miners are supposed to "censor" spam
80 2017-02-13 02:50:58 0|luke-jr|that's part of how the system works
81 2017-02-13 02:51:22 0|sipa|i agree with you, but only in its "development phase"... which may last a long time
82 2017-02-13 02:51:36 0|luke-jr|?
83 2017-02-13 02:51:41 0|sipa|everyone should cooperate to make the system as usable as possible while it is not perfect
84 2017-02-13 02:52:09 0|luke-jr|morcos: btw, Core wallet never reuses SPKs
85 2017-02-13 02:52:19 0|sipa|but eventually, it will need to work in a very-close-to-perfectly-rational environment
86 2017-02-13 02:52:38 0|luke-jr|sipa: okay, I think I agree on that.
87 2017-02-13 02:52:42 0|sipa|that doesn't mean that has to happen today
88 2017-02-13 02:53:58 0|luke-jr|IMO Lightning will help take a big step toward that
89 2017-02-13 02:54:06 0|sipa|maybe
90 2017-02-13 02:54:24 0|luke-jr|well, in theory since legit txs will probably drop in blockchain data usage vs spam
91 2017-02-13 02:54:50 0|luke-jr|hopefully that will increase the feerate of legit use beyond the point where spam is more economic
92 2017-02-13 05:42:33 0|bitcoin-git|[13bitcoin] 15rohundhar opened pull request #9750: Bloomfilter: parameter variables made constant (06master...06bloomVar) 02https://github.com/bitcoin/bitcoin/pull/9750
93 2017-02-13 13:03:57 0|bitcoin-git|[13bitcoin] 15DATSEC opened pull request #9751: Mini fix (06master...06patch-1) 02https://github.com/bitcoin/bitcoin/pull/9751
94 2017-02-13 13:12:43 0|jonasschnelli|^^ Hmm.. chaning changelogs seems wired
95 2017-02-13 14:28:07 0|jcorgan|is there a particular reason bitcoin.conf only allows IP parameters by address and not hostname/dns name, other than "it hasn't been written yet"?
96 2017-02-13 14:29:04 0|jonasschnelli|jcorgan: Yes. Probably. But I think we don't want addr-man to send around hostnames... in the config, yeah, fine.
97 2017-02-13 14:30:23 0|jcorgan|right, i'm only think of the conf file or cmd line, like proxy=, etc.
98 2017-02-13 14:30:27 0|jcorgan|*thinking
99 2017-02-13 16:45:47 0|Greybits|Hi! im not very smart and trying to learn more about segwit. to me it looks like lightning network is just a "man in the middle" proxy that can block transactions if they want to. why is lightning network good for crypto? who is bitfury? why do they have an office in washington dc? what do they do in north carolina by the military base?
100 2017-02-13 16:47:45 0|Chris_Stewart_5|Is there a BIP that talks about block time stamp requirements?
101 2017-02-13 16:50:54 0|Chris_Stewart_5|Looking at BIP113, it states we take the median time stamp for the last 11 blocks, but what stops the miner from egregiously lieing about that timestamp?
102 2017-02-13 17:12:17 0|BlueMatt|Chris_Stewart_5: there is a consensus rule that the MTP (median time past) of each block must progress
103 2017-02-13 17:12:33 0|BlueMatt|ie you acan never have a block which is older than the median-of-last-11 as per its timestamp
104 2017-02-13 17:15:56 0|Chris_Stewart_5|BlueMatt: Thanks
105 2017-02-13 19:12:56 0|bitcoin-git|[13bitcoin] 15ryanofsky opened pull request #9753: Add static_assert to prevent VARINT(<signed value>) (06master...06pr/varint-assert) 02https://github.com/bitcoin/bitcoin/pull/9753
106 2017-02-13 22:42:26 0|bitcoin-git|[13bitcoin] 15richardkiss opened pull request #9754: Change NULLFAIL => SIG_NULLFAIL. (06master...06feature/unify_sig_nullfail) 02https://github.com/bitcoin/bitcoin/pull/9754
107 2017-02-13 22:51:51 0|bitcoin-git|[13bitcoin] 15luke-jr opened pull request #9755: Bugfix: Qt/Options: Restore persistent "restart required" notice (06master...06bugfix_gui_opts_restartreq) 02https://github.com/bitcoin/bitcoin/pull/9755
108 2017-02-13 23:00:00 0|bitcoin-git|[13bitcoin] 15ryanofsky opened pull request #9756: Return error when importmulti called with invalid address. (06master...06pr/multiaddr) 02https://github.com/bitcoin/bitcoin/pull/9756