07:24 < wumpus> someone is trying to extort me, let it be clear that this is not an accepable email: "Hi all. I have the possibility of dos attacks, resistant to scoring ban, sending coins is not required. Can I get some coins? In particular, I would like 1 btc after demonstrating this, 1 btc after e

07:24 < wumpus> xplaining how to do it and how to how to protect from this, and 1 btc to anyone else not to talk about. Reply to this email if you are interested."

07:26 < bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bc121b0eb197...d932159f3467

07:26 < bitcoin-git> bitcoin/master ad04d1c Pieter Wuille: Always add default_witness_commitment with GBT client support

07:26 < bitcoin-git> bitcoin/master 95f4a03 Suhas Daftuar: [qa] Test getblocktemplate default_witness_commitment

07:26 < bitcoin-git> bitcoin/master d932159 Wladimir J. van der Laan: Merge #9189: Always add default_witness_commitment with GBT client support...

07:26 < luke-jr> wumpus: just ignore it IMO

07:27 < bitcoin-git> [bitcoin] laanwj closed pull request #9189: Always add default_witness_commitment with GBT client support (master...alwayscommit) https://github.com/bitcoin/bitcoin/pull/9189

07:27 < wumpus> luke-jr: yea I won't waste any time on it, but want to be public about it

07:28 < sipa> wumpus: i got the same ma

07:28 < sipa> ma

07:28 < sipa> maIL

07:28 < sipa> and i haven't responded

07:35 < rabidus> wow

07:59 < jonasschnelli> wumpus: got the same mail..

07:59 < jonasschnelli> won't reply to such "threats"...

08:07 < wumpus> happy it was not targeted specifically at least

08:26 < bitcoin-git> [bitcoin] laanwj opened pull request #9219: doc: Improve windows build instructions using Linux subsystem (master...2016_11_windows_build_update) https://github.com/bitcoin/bitcoin/pull/9219

08:33 < bitcoin-git> [bitcoin] fanquake closed pull request #8983: WIP: Log block height and size when received (master...ShowBlockHeightAndSizeWhenReceived) https://github.com/bitcoin/bitcoin/pull/8983

08:36 < fanquake> sipa any opinion on #8844 ?

08:36 < gribble> https://github.com/bitcoin/bitcoin/issues/8844 | Change sigops cost to sigops weight by jnewbery · Pull Request #8844 · bitcoin/bitcoin · GitHub

09:29 < gmaxwell> rusty: https://people.xiph.org/~greg/temp/fee_avail3.png looking pretty good the last couple days.

09:39 < bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d932159f3467...97ec6e5c9098

09:39 < bitcoin-git> bitcoin/master 5262a15 Daniel Cousens: tx_valid: re-order inputs to how they are encoded

09:39 < bitcoin-git> bitcoin/master 97ec6e5 MarcoFalke: Merge #9100: tx_valid: re-order inputs to how they are encoded...

09:39 < bitcoin-git> [bitcoin] MarcoFalke closed pull request #9100: tx_valid: re-order inputs to how they are encoded (master...testorder) https://github.com/bitcoin/bitcoin/pull/9100

10:26 < MarcoFalke> #9220 has same binaries, so fine to merge right now.

10:26 < gribble> https://github.com/bitcoin/bitcoin/issues/9220 | Refactor: Stop using namespace std (bitcoin-cli/-tx). by kallewoof · Pull Request #9220 · bitcoin/bitcoin · GitHub

10:36 < MarcoFalke> Just going to merge. Hope this is fine

10:36 < bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/97ec6e5c9098...e22f409f1888

10:36 < bitcoin-git> bitcoin/master 2f2625a Karl-Johan Alm: Removed using namespace std from bitcoin-cli/-tx and added std:: in appropriate places.

10:36 < bitcoin-git> bitcoin/master e22f409 MarcoFalke: Merge #9220: Refactor: Stop using namespace std (bitcoin-cli/-tx)....

10:37 < bitcoin-git> [bitcoin] MarcoFalke closed pull request #9220: Refactor: Stop using namespace std (bitcoin-cli/-tx). (master...no-using-ns) https://github.com/bitcoin/bitcoin/pull/9220

10:38 < MarcoFalke> #8776 looks good as well. Anyone other than jonas or me give another review on that one?

10:38 < gribble> https://github.com/bitcoin/bitcoin/issues/8776 | Wallet refactoring leading up to multiwallet by luke-jr · Pull Request #8776 · bitcoin/bitcoin · GitHub

10:40 < * luke-jr> wonders if we can ionice historical blocks somehow

10:52 < jonasschnelli> Would appreciate if someone reviews #9143 (~moveonly)

10:52 < gribble> https://github.com/bitcoin/bitcoin/issues/9143 | Refactor ZapWalletTxes to avoid layer vialotions by jonasschnelli · Pull Request #9143 · bitcoin/bitcoin · GitHub

10:53 < jonasschnelli> 9143 leads to CWallet* free CWalletDB which allows compiling CWalletDB without circular dependencies which would allow writing a tool that could create / manipulate wallets

10:53 < jonasschnelli> (next to a better class design)

10:56 < jonasschnelli> MarcoFalke: Yes. 8776 should go in as well.. maybe wait for a final (ut)ACK from wumpus

17:08 < bitcoin-git> [bitcoin] MarcoFalke closed pull request #8859: WIP: [qa] Add script to check for datadir compatibility between versions (master...Mf1610-qaCompat) https://github.com/bitcoin/bitcoin/pull/8859

17:13 < bitcoin-git> [bitcoin] MarcoFalke opened pull request #9221: [qa] maxuploadtarget: Get rid of duplicate code (master...Mf1611-qaMaxUploadDuplCode) https://github.com/bitcoin/bitcoin/pull/9221

19:18 < bitcoin-git> [bitcoin] sipa pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/e22f409f1888...76fec09d878d

19:19 < bitcoin-git> bitcoin/master d74e352 Cory Fields: net: Set feelers to disconnect at the end of the version message...

19:19 < bitcoin-git> bitcoin/master fedea8a Cory Fields: net: don't send any messages before handshake or after requested disconnect...

19:19 < bitcoin-git> bitcoin/master b7695c2 Cory Fields: net: No need to check individually for disconnection anymore

19:19 < bitcoin-git> [bitcoin] sipa closed pull request #9128: net: Decouple CConnman and message serialization (master...connman-send) https://github.com/bitcoin/bitcoin/pull/9128

19:20 < sipa> wumpus: opinion on #8610?

19:20 < gribble> https://github.com/bitcoin/bitcoin/issues/8610 | Share unused mempool memory with coincache by sipa · Pull Request #8610 · bitcoin/bitcoin · GitHub

19:30 < Chris_Stewart_5> To use CPFP you don't have to have sequence numbers set to 0xffffffff - 1 correct?

19:31 < sipa> indeed, it always works

20:27 < Jarid> Site where you can double your bitcoins in 48h [proof on site fb page] ----> https://coinfever.me/?ref=520be735339

20:28 < Lauda> #ban

20:29 < Chris_Stewart_5> So if you use the 'createrawtransactino' rpc, does this circumvent other flags you had set when starting bitcoind just as -walletrbf

20:36 < btcdrak> Looking for a review of #7562

20:36 < gribble> https://github.com/bitcoin/bitcoin/issues/7562 | Bump transaction version default to 2 by btcdrak · Pull Request #7562 · bitcoin/bitcoin · GitHub

20:50 < arubi> Chris_Stewart_5, if you're specifying the inputs, then you should also specify the sequence. afaik createrawtransaction is not a wallet function. you could createrawtransaction with just the outputs, then fundrawtransaction, and you'll get the inputs with opt-in rbf sequence

20:57 < Chris_Stewart_5> arubi: Thanks, yeah I was getting all maxed sequence numbers when using 'createrawtransaction' with -walletrbf set

20:58 < arubi> well if you're already specifying the inputs verbatim, just use "<txid>:<index>:<sequence>" sequence is base 10..

20:59 < arubi> or I guess, it takes json, add the "sequence" field. I just use bitcoin-tx :)

21:13 < Chris_Stewart_5> arubi: I'm using sendtoaddress rpc command with rbf and creates a tx with sequence number 4294967294 which is 0xffffffff - 1. This seems to be contradictory to BIP 125 which says the seqeuence number needs to be < 0xffffffff -1

21:14 < Chris_Stewart_5> with -walletrbf I should say

21:32 < arubi> Chris_Stewart_5, right, I use 2^32-3. I think that's what core does too

23:55 < bitcoin-git> [bitcoin] dooglus opened pull request #9222: Add 'subtractFeeFromAmount' option to 'fundrawtransaction'. (master...subtractFeeFromAmount-in-fundraw) https://github.com/bitcoin/bitcoin/pull/9222