12016-01-21T00:16:15 *** dcousens has quit IRC
22016-01-21T00:18:04 *** zooko|afk is now known as zooko
32016-01-21T00:18:23 *** dcousens has joined #bitcoin-core-dev
42016-01-21T00:25:36 *** drnet has quit IRC
52016-01-21T00:42:34 *** T23WS has joined #bitcoin-core-dev
62016-01-21T00:42:46 *** JackH has quit IRC
72016-01-21T00:44:41 *** randy-waterhouse has quit IRC
82016-01-21T00:59:26 *** ronbo has quit IRC
92016-01-21T01:06:52 *** brg444 has quit IRC
102016-01-21T01:10:16 *** T23WS_ has joined #bitcoin-core-dev
112016-01-21T01:12:26 *** T23WS has quit IRC
122016-01-21T01:14:10 *** maurris has joined #bitcoin-core-dev
132016-01-21T01:39:56 *** Chris_Stewart_5 has quit IRC
142016-01-21T01:48:38 <GitHub122> [bitcoin] s3v3nh4cks opened pull request #7385: Darknet (master...master) https://github.com/bitcoin/bitcoin/pull/7385
152016-01-21T01:49:18 <GitHub133> [bitcoin] s3v3nh4cks closed pull request #7385: Darknet (master...master) https://github.com/bitcoin/bitcoin/pull/7385
162016-01-21T01:53:11 *** Chris_Stewart_5 has joined #bitcoin-core-dev
172016-01-21T02:08:22 *** Ylbam has quit IRC
182016-01-21T02:13:46 <morcos> oops. thanks cfields...
192016-01-21T02:18:26 *** zooko has quit IRC
202016-01-21T02:18:57 *** brg444 has joined #bitcoin-core-dev
212016-01-21T02:20:03 *** zookolaptop has joined #bitcoin-core-dev
222016-01-21T02:22:20 *** harding_ is now known as harding
232016-01-21T02:41:20 *** T23WS has joined #bitcoin-core-dev
242016-01-21T02:44:36 *** T23WS_ has quit IRC
252016-01-21T02:46:25 *** T23WS_ has joined #bitcoin-core-dev
262016-01-21T02:47:58 *** T23WS has quit IRC
272016-01-21T02:54:55 *** frankenmint has joined #bitcoin-core-dev
282016-01-21T02:57:52 *** d_t has joined #bitcoin-core-dev
292016-01-21T02:59:41 *** frankenmint has quit IRC
302016-01-21T03:35:29 *** cryptojonathan has joined #bitcoin-core-dev
312016-01-21T03:36:57 *** cryptojonathan has quit IRC
322016-01-21T03:47:16 *** brg444 has quit IRC
332016-01-21T03:47:20 *** go1111111 has quit IRC
342016-01-21T03:47:51 *** guruvan has quit IRC
352016-01-21T03:47:52 *** BananaLotus has quit IRC
362016-01-21T03:48:57 *** drnet has joined #bitcoin-core-dev
372016-01-21T03:50:15 *** guruvan has joined #bitcoin-core-dev
382016-01-21T03:53:16 *** BananaLotus has joined #bitcoin-core-dev
392016-01-21T03:54:26 *** GAit has quit IRC
402016-01-21T03:54:53 *** GAit has joined #bitcoin-core-dev
412016-01-21T03:59:37 *** go1111111 has joined #bitcoin-core-dev
422016-01-21T04:05:40 *** dgenr8 has quit IRC
432016-01-21T04:08:26 *** drnet has quit IRC
442016-01-21T04:14:49 *** GAit has quit IRC
452016-01-21T04:17:03 *** dgenr8 has joined #bitcoin-core-dev
462016-01-21T04:17:19 *** randy-waterhouse has joined #bitcoin-core-dev
472016-01-21T04:23:26 *** go1111111 has quit IRC
482016-01-21T04:31:00 *** p15 has joined #bitcoin-core-dev
492016-01-21T05:04:48 *** T23WS_ has quit IRC
502016-01-21T05:32:53 <phantomcircuit> morcos, i no longer think there's a bug in there, but that there is something about the changes which made something else more obvious
512016-01-21T05:33:08 <phantomcircuit> possibly not hitting disk there made memory failures worse or something
522016-01-21T06:13:56 *** frankenmint has joined #bitcoin-core-dev
532016-01-21T06:18:48 *** frankenmint has quit IRC
542016-01-21T06:36:10 *** kang_ has joined #bitcoin-core-dev
552016-01-21T06:48:16 *** murch has joined #bitcoin-core-dev
562016-01-21T06:59:36 *** Ylbam has joined #bitcoin-core-dev
572016-01-21T07:07:12 *** arowser has quit IRC
582016-01-21T07:07:45 *** arowser has joined #bitcoin-core-dev
592016-01-21T07:09:01 *** Alopex has quit IRC
602016-01-21T07:10:07 *** Alopex has joined #bitcoin-core-dev
612016-01-21T07:27:46 *** PaulCapestany has quit IRC
622016-01-21T07:30:19 *** go1111111 has joined #bitcoin-core-dev
632016-01-21T07:31:40 *** PaulCapestany has joined #bitcoin-core-dev
642016-01-21T08:01:09 <GitHub71> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b92ea98503e6...ae2db67feec2
652016-01-21T08:01:10 <GitHub71> bitcoin/master df6e8e1 Jonas Schnelli: [Qt] rename "amount" to "requested amount" in receive coins table
662016-01-21T08:01:11 <GitHub71> bitcoin/master ae2db67 Jonas Schnelli: Merge #7383: [Qt] rename "amount" to "requested amount" in receive coins table...
672016-01-21T08:01:19 <GitHub34> [bitcoin] jonasschnelli closed pull request #7383: [Qt] rename "amount" to "requested amount" in receive coins table (master...2016/01/qt_req_amount) https://github.com/bitcoin/bitcoin/pull/7383
682016-01-21T08:14:47 *** frankenmint has joined #bitcoin-core-dev
692016-01-21T08:19:24 *** frankenmint has quit IRC
702016-01-21T08:19:51 *** murch has quit IRC
712016-01-21T08:35:13 *** Arnavion has quit IRC
722016-01-21T08:35:17 *** Arnavion3 has joined #bitcoin-core-dev
732016-01-21T08:35:21 *** Arnavion3 is now known as Arnavion
742016-01-21T08:37:41 *** JackH has joined #bitcoin-core-dev
752016-01-21T08:41:17 *** dayZh has joined #bitcoin-core-dev
762016-01-21T08:43:39 *** BashCo has quit IRC
772016-01-21T09:05:12 *** Guyver2 has joined #bitcoin-core-dev
782016-01-21T09:05:18 *** da2ce7_ is now known as da2ce7
792016-01-21T09:05:53 *** BashCo has joined #bitcoin-core-dev
802016-01-21T09:10:31 <wumpus> cfields, something w/ travis is strange on the 0.10 branch, https://travis-ci.org/bitcoin/bitcoin/jobs/103586898 . fukuchi.org is failing (download for qrencode) but it's not using the fallback - there isn't even an error for bitcoincore.org/dev.bitcoincore.org
812016-01-21T09:11:55 *** AtashiCon has quit IRC
822016-01-21T09:12:17 *** Arnavion has quit IRC
832016-01-21T09:24:10 *** Guyver2 has quit IRC
842016-01-21T09:40:43 *** laurentmt has joined #bitcoin-core-dev
852016-01-21T09:40:52 *** laurentmt has quit IRC
862016-01-21T09:49:16 *** anondran has joined #bitcoin-core-dev
872016-01-21T09:53:10 *** frankenmint has joined #bitcoin-core-dev
882016-01-21T09:58:05 *** frankenmint has quit IRC
892016-01-21T09:59:57 *** anondran has left #bitcoin-core-dev
902016-01-21T10:06:07 *** _sf_ has joined #bitcoin-core-dev
912016-01-21T10:06:42 *** frankenmint has joined #bitcoin-core-dev
922016-01-21T10:08:35 *** da2ce7_mobile has quit IRC
932016-01-21T10:08:35 *** da2ce7 has quit IRC
942016-01-21T10:08:50 *** da2ce7 has joined #bitcoin-core-dev
952016-01-21T10:09:55 *** da2ce7_mobile has joined #bitcoin-core-dev
962016-01-21T10:13:51 <GitHub174> [bitcoin] laanwj opened pull request #7386: Add option `-permitrbf` to set transaction replacement policy (master...2016_01_disable_opt_in_rbf) https://github.com/bitcoin/bitcoin/pull/7386
972016-01-21T10:19:15 <btcdrak> wumpus: great
982016-01-21T10:22:17 <CodeShark> I don't get it, wumpus
992016-01-21T10:22:21 <CodeShark> where's the GetBoolArg?
1002016-01-21T10:30:25 *** da2ce7 has quit IRC
1012016-01-21T10:30:25 *** da2ce7_mobile has quit IRC
1022016-01-21T10:35:48 *** da2ce7 has joined #bitcoin-core-dev
1032016-01-21T10:36:47 *** da2ce7_mobile has joined #bitcoin-core-dev
1042016-01-21T10:56:56 *** JackH has quit IRC
1052016-01-21T11:03:38 *** JackH has joined #bitcoin-core-dev
1062016-01-21T11:10:05 *** MarcoFalke has joined #bitcoin-core-dev
1072016-01-21T11:14:11 *** dayZh has quit IRC
1082016-01-21T11:18:49 *** kang_ has quit IRC
1092016-01-21T11:26:24 *** ronbo has joined #bitcoin-core-dev
1102016-01-21T11:27:42 *** GAit has joined #bitcoin-core-dev
1112016-01-21T11:30:46 *** MarcoFalke has quit IRC
1122016-01-21T11:32:28 *** dayZh has joined #bitcoin-core-dev
1132016-01-21T11:36:26 *** dayZh has quit IRC
1142016-01-21T11:37:04 <GitHub109> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ae2db67feec2...6f7841d54558
1152016-01-21T11:37:05 <GitHub109> bitcoin/master b768108 Wladimir J. van der Laan: Add option `-permitrbf` to set transaction replacement policy...
1162016-01-21T11:37:05 <GitHub109> bitcoin/master 6f7841d Wladimir J. van der Laan: Merge #7386: Add option `-permitrbf` to set transaction replacement policy...
1172016-01-21T11:37:14 <GitHub28> [bitcoin] laanwj closed pull request #7386: Add option `-permitrbf` to set transaction replacement policy (master...2016_01_disable_opt_in_rbf) https://github.com/bitcoin/bitcoin/pull/7386
1182016-01-21T11:37:26 *** paveljanik has joined #bitcoin-core-dev
1192016-01-21T11:38:45 *** dagurval has quit IRC
1202016-01-21T11:42:42 *** randy-waterhouse has quit IRC
1212016-01-21T11:48:55 <GitHub53> [bitcoin] laanwj pushed 1 new commit to 0.12: https://github.com/bitcoin/bitcoin/commit/da83ecd45478367e5388c58fe29a4a8d72f7f1cd
1222016-01-21T11:48:56 <GitHub53> bitcoin/0.12 da83ecd Wladimir J. van der Laan: Add option `-permitrbf` to set transaction replacement policy...
1232016-01-21T11:51:41 *** drnet has joined #bitcoin-core-dev
1242016-01-21T12:01:49 *** drnet has quit IRC
1252016-01-21T12:18:55 *** grassass has joined #bitcoin-core-dev
1262016-01-21T12:20:07 <GitHub75> [bitcoin] sipa opened pull request #7387: Get rid of inaccurate ScriptSigArgsExpected (0.12...inaccscriptargs) https://github.com/bitcoin/bitcoin/pull/7387
1272016-01-21T12:25:34 <GitHub24> [bitcoin] sipa reopened pull request #6597: Chainparams: Don't check the genesis block (master...chainparams-genesis-no-check-0.12.99) https://github.com/bitcoin/bitcoin/pull/6597
1282016-01-21T12:28:21 <dcousens> how often does bitcoind flush to the .dat files?
1292016-01-21T12:31:59 *** Alina-malina has joined #bitcoin-core-dev
1302016-01-21T12:34:23 <jonasschnelli> dcousens: relevant part: https://github.com/bitcoin/bitcoin/blob/6f7841d5455832bceeb0b1375baef772099d5595/src/main.cpp#L2228
1312016-01-21T12:36:41 *** Alina-malina has left #bitcoin-core-dev
1322016-01-21T12:38:40 <dcousens> ta
1332016-01-21T12:40:29 <dcousens> jonasschnelli: how does that work with things like txindex?
1342016-01-21T12:40:39 <dcousens> does the DiskPos sometimes refer to a memory location?
1352016-01-21T12:41:01 <dcousens> aka is there a memory map for things not yet written etc?
1362016-01-21T12:42:03 *** GAit has quit IRC
1372016-01-21T12:50:45 *** dayZh has joined #bitcoin-core-dev
1382016-01-21T12:55:22 <GitHub37> [bitcoin] sandakersmann opened pull request #7388: Set `-permitrbf` to false (master...patch-1) https://github.com/bitcoin/bitcoin/pull/7388
1392016-01-21T13:13:35 *** Thireus has joined #bitcoin-core-dev
1402016-01-21T13:30:42 *** Thireus1 has joined #bitcoin-core-dev
1412016-01-21T13:33:26 *** Thireus has quit IRC
1422016-01-21T13:39:41 *** dagurval has joined #bitcoin-core-dev
1432016-01-21T13:40:32 *** dayZh has quit IRC
1442016-01-21T13:45:53 *** Yoghur114 has quit IRC
1452016-01-21T13:52:30 *** dayZh has joined #bitcoin-core-dev
1462016-01-21T13:54:31 *** _sf_ has left #bitcoin-core-dev
1472016-01-21T13:57:38 *** dayZh has quit IRC
1482016-01-21T14:00:15 *** Thireus1 has quit IRC
1492016-01-21T14:02:31 *** Thireus has joined #bitcoin-core-dev
1502016-01-21T14:03:13 *** p15x has joined #bitcoin-core-dev
1512016-01-21T14:11:10 *** dayZh has joined #bitcoin-core-dev
1522016-01-21T14:11:48 *** treehug88 has joined #bitcoin-core-dev
1532016-01-21T14:17:57 *** p15x has quit IRC
1542016-01-21T14:22:36 *** p15 has quit IRC
1552016-01-21T14:27:28 *** berndj has quit IRC
1562016-01-21T14:30:27 *** berndj has joined #bitcoin-core-dev
1572016-01-21T14:32:35 *** dcousens has quit IRC
1582016-01-21T14:36:23 *** GAit has joined #bitcoin-core-dev
1592016-01-21T14:42:40 <GitHub21> [bitcoin] laanwj closed pull request #7005: Add MAINTAINERS file, a user guide to code subsystems (master...2015_maintainer) https://github.com/bitcoin/bitcoin/pull/7005
1602016-01-21T15:00:31 *** zookolaptop has quit IRC
1612016-01-21T15:20:13 *** GAit has quit IRC
1622016-01-21T15:20:22 *** GAit has joined #bitcoin-core-dev
1632016-01-21T15:21:14 <GitHub180> [bitcoin] NicolasDorier closed pull request #7190: Performance fix for #6312 (master...sequencenumbers) https://github.com/bitcoin/bitcoin/pull/7190
1642016-01-21T15:54:28 *** GAit has quit IRC
1652016-01-21T15:54:59 *** GAit has joined #bitcoin-core-dev
1662016-01-21T16:00:18 *** jtimon has joined #bitcoin-core-dev
1672016-01-21T16:21:38 *** zookolaptop has joined #bitcoin-core-dev
1682016-01-21T16:25:10 *** dayZh has quit IRC
1692016-01-21T16:29:55 *** ronbo has quit IRC
1702016-01-21T16:37:00 *** murch has joined #bitcoin-core-dev
1712016-01-21T16:40:35 *** paveljanik has quit IRC
1722016-01-21T17:00:12 *** Amnez777 has quit IRC
1732016-01-21T17:00:17 *** Tasoshi has joined #bitcoin-core-dev
1742016-01-21T17:00:43 *** BashCo has quit IRC
1752016-01-21T17:01:35 <BlueMatt> wangchun: regarding opt-in rbf and 0conf....be careful, you've been sold a pack of lies. EVERY company which I'm aware of which currently accepts 0conf has stated publicly that opt-in rbf does not impact them
1762016-01-21T17:02:01 <BlueMatt> wangchun: many wallet creators have been asking for it because it makes paying fees much, much easier
1772016-01-21T17:02:25 <BlueMatt> wangchun: the ONLY people who are complaining about opt-in rbf are on reddit
1782016-01-21T17:06:12 *** Nagato has joined #bitcoin-core-dev
1792016-01-21T17:06:35 *** Nagato is now known as Guest30744
1802016-01-21T17:19:37 *** Guest30744 has quit IRC
1812016-01-21T17:23:50 *** orbiting has joined #bitcoin-core-dev
1822016-01-21T17:28:00 <BlueMatt> wangchun: for example, blockcypher: https://twitter.com/BlockCypher/status/670334879565922304
1832016-01-21T17:29:57 *** BashCo has joined #bitcoin-core-dev
1842016-01-21T17:32:52 *** trippysalmon has joined #bitcoin-core-dev
1852016-01-21T17:35:24 <instagibbs> BlueMatt, I've been quoting that thread, it'd be a good idea to get more on the record to explictly rebut the FUD
1862016-01-21T17:38:54 <BlueMatt> instagibbs: lol, eric vorhees, in a long post talking about why availability of 0conf is a good idea, a guy who is explicitly incredibly pissed off at core for various reasons, said this: "The current Bitcoin implementation, in my opinion, offered a very elegant balance between instant settlement and security. Zero-conf was always an option for merchants or individuals it came with risk, but it was in many cases manageable, and bri
1872016-01-21T17:38:55 <BlueMatt> lliant companies like BlockCypher rose up to make it even more manageable. For those who didn't want any risk, they could decide to wait for one or several confirmations. Neither group infringed on the other SatoshiDICE was able to take on the risk of zero-conf while other firms decided to wait for blocks. Mutual harmony was enjoyed. "
1882016-01-21T17:39:02 <BlueMatt> links is https://shapeshift.io/site/blog/2015/12/01/note-ceo-erik-voorhees-appeal-zero-conf
1892016-01-21T17:40:47 *** laurentmt has joined #bitcoin-core-dev
1902016-01-21T17:44:20 *** laurentmt has quit IRC
1912016-01-21T18:00:14 *** Amnez777 has joined #bitcoin-core-dev
1922016-01-21T18:03:23 *** Amnez777 has quit IRC
1932016-01-21T18:03:24 *** Amnez777 has joined #bitcoin-core-dev
1942016-01-21T18:10:51 <Tasoshi> isn't this a bit off topic?
1952016-01-21T18:14:17 *** Arnavion has joined #bitcoin-core-dev
1962016-01-21T18:14:23 *** AtashiCon has joined #bitcoin-core-dev
1972016-01-21T18:18:05 *** GAit has quit IRC
1982016-01-21T18:18:52 *** GAit has joined #bitcoin-core-dev
1992016-01-21T18:22:38 *** GAit has quit IRC
2002016-01-21T18:25:02 *** GAit has joined #bitcoin-core-dev
2012016-01-21T18:31:27 <cfields> wumpus: i'll take a look
2022016-01-21T18:32:40 <haakonn> oops, sorry, my message was meant for a different channel ...
2032016-01-21T18:34:06 <warren> wangchun: another example, Bread Wallet's Aaron Voisine thinks opt-in RBF is great.
2042016-01-21T18:34:22 *** dayZh has joined #bitcoin-core-dev
2052016-01-21T18:38:51 *** dayZh has quit IRC
2062016-01-21T18:50:24 *** kangx has joined #bitcoin-core-dev
2072016-01-21T18:51:08 *** challisto has quit IRC
2082016-01-21T18:53:06 *** GAit has quit IRC
2092016-01-21T19:05:07 *** kangx has quit IRC
2102016-01-21T19:09:15 <jonasschnelli> phantomcircuit: re RBF: I think if there are merchants (example third world) that accept 0-confs over a standard wallet (Android Wallet). These guys would suffer a higher chance to get double spended
2112016-01-21T19:10:08 <phantomcircuit> jonasschnelli, anybody using BIP70 payment protocol is already at extreme risk accepting unconfirmed transactions
2122016-01-21T19:10:09 *** GAit has joined #bitcoin-core-dev
2132016-01-21T19:10:28 <phantomcircuit> because the transaction is relayed directly to the merchant, timing attacks are trivial
2142016-01-21T19:10:32 <phantomcircuit> (and very effective)
2152016-01-21T19:10:44 <BlueMatt> jonasschnelli: such people are already trivially attackable using petertodd's rbf tools
2162016-01-21T19:10:53 <phantomcircuit> oh AND spv clients have a hard time guessing what fees are required to be included in the next block
2172016-01-21T19:11:00 <BlueMatt> jonasschnelli: more importantly, those wallets can be easily upgraded to fix that issue
2182016-01-21T19:11:06 <jonasschnelli> BlueMatt : totally agree with this. It just makes double spending more easy...
2192016-01-21T19:11:18 <BlueMatt> jonasschnelli: my point is it /doesnt/ make double-spending easier
2202016-01-21T19:11:25 <jonasschnelli> But maybe RBF is then a sign for them to "move on" and use better tools (then just a wallet as merchant)
2212016-01-21T19:11:32 <BlueMatt> it does make double-spending easier against people like blockcypher, but they know this and have already adapted
2222016-01-21T19:11:33 <phantomcircuit> jonasschnelli, the situation is such that unconfirmed transactions cannot be safely accepted by anybody unless they have the ability to reverse their side of the transaction after the fact
2232016-01-21T19:11:44 <phantomcircuit> jonasschnelli, which actually describes most online merchants
2242016-01-21T19:11:56 <BlueMatt> jonasschnelli: and some wallets already are being fixed - they're displaying opt-in rbf as 'pending' instead of 'unconfirmed'
2252016-01-21T19:12:01 <BlueMatt> its a much clearer indication
2262016-01-21T19:12:22 <BlueMatt> long before the network is even using opt-in rbf wallets can be ready
2272016-01-21T19:12:25 <jonasschnelli> Wallets maybe also want to show a "icon" "warning" in case a RBF transaction is amount the 0-conf incommint wtxs
2282016-01-21T19:13:01 <BlueMatt> yes, I know at least breadwallet is doing something like that
2292016-01-21T19:13:05 <BlueMatt> I dont remeber the actual wording
2302016-01-21T19:13:10 <BlueMatt> something like pending or whatever
2312016-01-21T19:17:06 <phantomcircuit> BlueMatt, they probably should have already been doing that since the signaling is to set nSequence < MAX_INT
2322016-01-21T19:17:17 <phantomcircuit> which has always signaled the transaction was available to be replaced
2332016-01-21T19:18:08 <gmaxwell> jonasschnelli: wow, accepting 0conf in android wallet is very unsafe... it can't even tell if the inputs ever existed to begin with. Send android wallet an unconfiremd 21 million btc payment, and it happily shows it as unconfirmed.
2342016-01-21T19:18:41 <BlueMatt> gmaxwell: actually it can...bitcoinj traces back all unconfirmed transactions to things in the chain
2352016-01-21T19:18:49 <BlueMatt> and validates they are all non-locktime'd, etc
2362016-01-21T19:18:51 <sdaftuar> phantomcircuit: if locktime is in the past, then nSequence < MAX_INT seems safe
2372016-01-21T19:19:15 <gmaxwell> BlueMatt: Android wallet doesn't. I've tested this myself.
2382016-01-21T19:19:23 <BlueMatt> gmaxwell: huh? It used to....wtf
2392016-01-21T19:19:32 <BlueMatt> sdaftuar: but, this means bitcoinj already has logic to hide non-final txn: it should be re-used for opt-in rbf :)
2402016-01-21T19:19:37 <phantomcircuit> sdaftuar, historically nSequence could be used to replace a transaction regardless of locktime
2412016-01-21T19:19:51 <phantomcircuit> (unless im misremembering wildly)
2422016-01-21T19:19:57 <sdaftuar> BlueMatt: i was actually looking at breadwallet's code :)
2432016-01-21T19:19:58 <gmaxwell> (it checks locktime, and hides non-final tx, but will happily show transactions with never-existed inputs)
2442016-01-21T19:20:15 <sdaftuar> BlueMatt: the problem SPV wallets have is needing to download unconfirmed ancestors i think?
2452016-01-21T19:20:26 <BlueMatt> gmaxwell: ahhh...someone should fix that
2462016-01-21T19:20:35 <sdaftuar> voisine opened an issue some time ago that is related to that, i think
2472016-01-21T19:20:37 <BlueMatt> sdaftuar: they've always needed to do that
2482016-01-21T19:20:58 <BlueMatt> sdaftuar: you can otherwise give them a) an invalid transaction, b) a transaction that is locked for 100 years because of a dependent
2492016-01-21T19:21:11 <BlueMatt> hence bitcoinj's behavior
2502016-01-21T19:21:16 <BlueMatt> though apparently that part is incomplete
2512016-01-21T19:21:30 <sdaftuar> BlueMatt: agreed that its necessary, not sure everyone has implemented though
2522016-01-21T19:21:40 <sdaftuar> voisine specifically requested a feature to make that easier
2532016-01-21T19:21:42 <BlueMatt> indeed, but we arent changing anything wrt that
2542016-01-21T19:22:10 <sdaftuar> #7237
2552016-01-21T19:22:23 <BlueMatt> we could have a p2p command that does that automatically (ie gives you all deps and merkle paths for them, buttttt)
2562016-01-21T19:22:35 <BlueMatt> ahh, heh, ok
2572016-01-21T19:22:37 <sdaftuar> i think the patch to that would be pretty small?
2582016-01-21T19:22:44 <BlueMatt> yea, it needs to also give merkle paths to the ones in the blocks
2592016-01-21T19:22:51 <BlueMatt> sadly, this is probably a bad dos vector, I'd think
2602016-01-21T19:23:06 <BlueMatt> maybe even worse than the bloom filtering code itself :(
2612016-01-21T19:23:08 <BlueMatt> though not likely
2622016-01-21T19:24:02 <sdaftuar> BlueMatt: i haven't given any thought to that aspect. just highlighting the issue so we can decide if it's worth doing, toward the goal of helping wallets do things the best way possible.
2632016-01-21T19:25:26 <BlueMatt> sdaftuar: yea, I wouldnt have a huge issue with doing it for clients that both request it and bitcoind instances that have bloom filtering enabled
2642016-01-21T19:25:34 <BlueMatt> its probably not hugely worse than the existing dos vectors, sooooo
2652016-01-21T19:26:17 <BlueMatt> still, we need reasonable communication like "is bitcoin core eating your hdd? are you seeing a lot of outbound traffic and no inbound? are your bitcoin core ping times a few seconds? disable bloom filtering"
2662016-01-21T19:26:43 <BlueMatt> oh, and fix the "mempool" command, but....whatever
2672016-01-21T19:26:53 <sdaftuar> BlueMatt: ok, i might take a look at doing it. is there any significant downside to just changing the existing behavior? i coulnd't find any BIP referencing the reseponse to the mempool command given a bloom filter
2682016-01-21T19:27:09 <sdaftuar> BlueMatt: and seems like SPV wallets have to check each response anyway...
2692016-01-21T19:27:43 <BlueMatt> sdaftuar: i mean if you want to make it easy, there should just be a command like "prove to me this transaction isnt bogus"
2702016-01-21T19:27:56 <BlueMatt> which gives you the dep chain and spv proofs that the leaves are included
2712016-01-21T19:27:58 <sdaftuar> oh new p2p command
2722016-01-21T19:28:05 <BlueMatt> thats how /I/ would do it
2732016-01-21T19:28:08 <BlueMatt> but...meh, as you wish :)
2742016-01-21T19:28:12 <sdaftuar> maybe that's cleaner, but seems like more work :)
2752016-01-21T19:28:22 <BlueMatt> a bit more, but not much, I'd think
2762016-01-21T19:28:30 <BlueMatt> and it clearly tells spv client implementors what to do
2772016-01-21T19:29:22 <sdaftuar> fair point
2782016-01-21T19:29:38 <BlueMatt> others will likely not like this idea since they want to deprecate the bloom filtering stuff entirely
2792016-01-21T19:29:56 <BlueMatt> though adding a new command that is like "this is a dos risk, but people who want to serve spv clients should give them this info"
2802016-01-21T19:30:16 <sdaftuar> that might also make it easy/easier to rate limit, i'd think
2812016-01-21T19:30:23 <BlueMatt> ideally we could make it an rpc and spv wallet implements would connect to their central server and get that info from them, but...whatever
2822016-01-21T19:30:29 <BlueMatt> sdaftuar: quite possible, indeed
2832016-01-21T19:30:53 <BlueMatt> but, more importantly, if we pull out the bloom filtering shit and replace it with something sane that serves a similar purpose, we dont have to touch this rpc :)
2842016-01-21T19:31:04 <BlueMatt> ehh, msg
2852016-01-21T19:31:14 <gmaxwell> getting dependency chains is incompatible with pruning-- or even without running with a huge high overhead index, in any case.
2862016-01-21T19:31:56 <BlueMatt> gmaxwell: you dont actually need -txindex right now, though
2872016-01-21T19:32:03 <BlueMatt> gmaxwell: its a huge dos risk either way :)
2882016-01-21T19:32:18 <BlueMatt> gmaxwell: (we know when a utxo was created, so we can load that block, you just cant prune)
2892016-01-21T19:33:58 *** GAit has quit IRC
2902016-01-21T19:48:36 *** ronbo has joined #bitcoin-core-dev
2912016-01-21T19:57:12 *** zookolaptop has quit IRC
2922016-01-21T19:57:46 *** kangx has joined #bitcoin-core-dev
2932016-01-21T20:04:17 *** brg444 has joined #bitcoin-core-dev
2942016-01-21T20:04:28 *** wumpus has quit IRC
2952016-01-21T20:06:31 *** e4xit has joined #bitcoin-core-dev
2962016-01-21T20:09:47 *** Dizzle has joined #bitcoin-core-dev
2972016-01-21T20:11:03 *** wumpus has joined #bitcoin-core-dev
2982016-01-21T20:11:13 *** kangx has quit IRC
2992016-01-21T20:14:25 *** kangx has joined #bitcoin-core-dev
3002016-01-21T20:17:14 *** orbiting has quit IRC
3012016-01-21T20:24:36 *** Guyver2 has joined #bitcoin-core-dev
3022016-01-21T20:24:41 *** GAit has joined #bitcoin-core-dev
3032016-01-21T20:29:02 *** aspect_ has joined #bitcoin-core-dev
3042016-01-21T20:34:36 *** JackH has quit IRC
3052016-01-21T20:37:37 <instagibbs> Sorry if this is pure C++ q: I'm adding a .h/.cpp file pair for additional rpc stuff, and I can't get the linker to link and external library I'm calling inside that file(undefined references to libevent functions). I figure I'm missing something super obvious but don't really understand largish automake files.
3062016-01-21T20:38:29 <instagibbs> I figure I need to link -levent somewhere in addition to bitcoind/bitcoin-cli in the Makefile.am, but noob so.
3072016-01-21T20:40:19 *** laurentmt has joined #bitcoin-core-dev
3082016-01-21T20:40:55 *** kangx has quit IRC
3092016-01-21T20:41:18 *** simpleirc1 has joined #bitcoin-core-dev
3102016-01-21T20:43:11 *** arowser has quit IRC
3112016-01-21T20:43:37 *** arowser has joined #bitcoin-core-dev
3122016-01-21T20:44:05 <treehug88> instagibbs look at configure.ac
3132016-01-21T20:44:57 <GitHub142> [bitcoin] MarcoFalke closed pull request #7374: [init] Add transaction fee warnings (master...Mf1601-init-fee-warn) https://github.com/bitcoin/bitcoin/pull/7374
3142016-01-21T20:46:25 *** brg444 has quit IRC
3152016-01-21T20:47:01 <cfields> wangchun: looks like i forgot to ping you here and you're probably gone now. but: ping!
3162016-01-21T20:52:30 *** zookolaptop has joined #bitcoin-core-dev
3172016-01-21T20:53:14 *** treehug88 has quit IRC
3182016-01-21T20:58:01 *** dayZh has joined #bitcoin-core-dev
3192016-01-21T21:02:48 *** dayZh has quit IRC
3202016-01-21T21:04:28 *** treehug88 has joined #bitcoin-core-dev
3212016-01-21T21:12:15 *** cjamt has joined #bitcoin-core-dev
3222016-01-21T21:24:16 *** frankenmint has quit IRC
3232016-01-21T21:24:51 *** frankenmint has joined #bitcoin-core-dev
3242016-01-21T21:28:47 *** zookolaptop has quit IRC
3252016-01-21T21:29:16 *** frankenmint has quit IRC
3262016-01-21T21:29:21 *** zookolaptop has joined #bitcoin-core-dev
3272016-01-21T21:36:39 *** maurris has left #bitcoin-core-dev
3282016-01-21T21:39:49 *** e4xit has quit IRC
3292016-01-21T21:40:41 *** murch has quit IRC
3302016-01-21T21:44:15 *** laurentmt has quit IRC
3312016-01-21T21:50:02 *** achow101 has joined #bitcoin-core-dev
3322016-01-21T21:53:52 *** JackH has joined #bitcoin-core-dev
3332016-01-21T22:00:26 *** achow101 has quit IRC
3342016-01-21T22:04:14 *** treehug88 has quit IRC
3352016-01-21T22:10:01 *** randy-waterhouse has joined #bitcoin-core-dev
3362016-01-21T22:25:22 *** frankenmint has joined #bitcoin-core-dev
3372016-01-21T22:30:19 *** frankenmint has quit IRC
3382016-01-21T22:39:29 *** Guyver2 has quit IRC
3392016-01-21T22:50:13 *** proslogion has joined #bitcoin-core-dev
3402016-01-21T22:53:35 *** trippysalmon has quit IRC
3412016-01-21T23:12:53 *** dayZh has joined #bitcoin-core-dev
3422016-01-21T23:15:56 *** belcher has joined #bitcoin-core-dev
3432016-01-21T23:17:20 *** dayZh has quit IRC
3442016-01-21T23:22:25 <gmaxwell> oh we're still doing this:
3452016-01-21T23:22:26 <gmaxwell> 2016-01-21 23:21:56 ERROR: Read: Failed to open file /home/gmaxwell/.bitcoin/banlist.dat
3462016-01-21T23:22:29 <gmaxwell> 2016-01-21 23:21:56 Invalid or missing banlist.dat; recreating
3472016-01-21T23:22:47 <phantomcircuit> gmaxwell, yup
3482016-01-21T23:23:09 *** brg444 has joined #bitcoin-core-dev
3492016-01-21T23:25:16 *** Thireus has quit IRC
3502016-01-21T23:26:34 *** frankenmint has joined #bitcoin-core-dev
3512016-01-21T23:29:45 *** GAit has quit IRC
3522016-01-21T23:31:20 <Ylbam> a segwit core dev would be welcome on core slack to answer a few questions
3532016-01-21T23:31:27 <Ylbam> https://bitcoincore.slack.com/archives/general/p1453418779022985
3542016-01-21T23:32:34 *** GAit has joined #bitcoin-core-dev
3552016-01-21T23:36:23 *** Dizzle has quit IRC
3562016-01-21T23:37:03 *** BashCo_ has joined #bitcoin-core-dev
3572016-01-21T23:37:05 *** dcousens has joined #bitcoin-core-dev
3582016-01-21T23:40:14 *** BashCo has quit IRC
3592016-01-21T23:42:56 <randy-waterhouse> slack core require registration with slack?
3602016-01-21T23:44:15 *** GAit has quit IRC
3612016-01-21T23:45:23 <Ylbam> slack.bitcoincore.org I think
3622016-01-21T23:46:24 *** JackH has quit IRC
3632016-01-21T23:46:34 <randy-waterhouse> and their servers are US-based I suppose?
3642016-01-21T23:47:10 *** mbmott has joined #bitcoin-core-dev
3652016-01-21T23:48:14 <randy-waterhouse> nm, I jsut wandered into their thicket of user policies ... looks like an avoidable minefield
3662016-01-21T23:51:01 <Ylbam> we are sending people here for more technical answers
3672016-01-21T23:51:32 <Ylbam> or there #segwit-dev