< meshcollider> promag: perhaps nothing since #11281 ?
< gribble> https://github.com/bitcoin/bitcoin/issues/11281 | Avoid permanent cs_main/cs_wallet lock during RescanFromTime by jonasschnelli · Pull Request #11281 · bitcoin/bitcoin · GitHub
< meshcollider> haven't tested though
< jtimon> should we log blocks that are consensus invalid but still have valid proof of work in a special way? (ie with admiration marks and with the full hex so that is not so easily forgotten)
< jtimon> is there an archive of invalid blocks that have valid pow ? (new bhardfork wave aside, perhaps)
< jtimon> disclaimer (perhaps offtopic): this comes to mind because in the case of signed blocks in elements, the result shouldn't be a some mining pool be mocked if it's one or people getting worried about an attack on spv nodes if there are many in a row, your node should simply stop to see what's up with the signers
< jtimon> s/stop to see/stop. To see/
< jtimon> last offtopic comment, the code is here for elements-0.14 https://github.com/ElementsProject/elements/pull/324 but perhaps I could do the logging thing for bitcoin if there's interest
< aj> jtimon: getchaintips gives you a log of invalid blocks with plausible proof of work, doesn't it?
< jtimon> aj: I don't know, I will look into that, thanks
< aj> jtimon: i see 00000000000000000014c1ee89b61a84e3e30dd9b2c78c9916d323a2775bc613 00000000000000000027894f0969c2f79a6cdb1231750e048effbd17c88da431 and 000000000000000000571c2b98a090c15774cb7400bd4a50b160d488d14055d0 as fairly recent blocks marked invalid...
< jtimon> and do they pass validation:CheckBlockHeader() ?
< jtimon> let me rephrase, what do you mean by "plausible" proof of work? isn't that rule a boolean like any other? you pass it or not?
< jtimon> anyway, thanks again
< aj> jtimon: "plausible" == "i'm not sure what's going on" :) you're probably right
< jtimon> either way I thank you for making me grep getchaintips, I didn't know that existed :)
< aj> hmm, maybe leaving debug=mempool on was a mistake
< aj> okay, that first block came through via a compactblock, got reconstructed successfully, but was then considered invalid due to CheckTxInputs, inputs missing/spent
< jtimon> I feel ashamed of not knowing getchaintips
< aj> jtimon: i feel glad i could tell you something new, though :)
< jtimon> aj: I mean, the shame passes fast, now I'm happy I learned something new potentially useful for an explorer
< bitcoin-git> [bitcoin] practicalswift closed pull request #12781: Don't test size_t variables for negative values (master...–size_t) https://github.com/bitcoin/bitcoin/pull/12781
< jojeyh> for segwit txs, when the client list transactions should there be a field for "wtxid" ?
< jojeyh> like when i enter 'bitcoin-cli listtransactions'
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0a018431c447...f4353daa61f4
< bitcoin-git> bitcoin/master 3a0f8d7 practicalswift: Qt: Remove unused method setupAmountWidget(...)
< bitcoin-git> bitcoin/master f4353da Wladimir J. van der Laan: Merge #12779: Qt: Remove unused method setupAmountWidget(...)...
< bitcoin-git> [bitcoin] laanwj closed pull request #12779: Qt: Remove unused method setupAmountWidget(...) (master...remove-unused-function-setupAmountWidget) https://github.com/bitcoin/bitcoin/pull/12779
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f4353daa61f4...17b246f4b3f5
< bitcoin-git> bitcoin/master 342fb80 MarcoFalke: qt: Avoid resetting on resetguisettigs=0
< bitcoin-git> bitcoin/master 17b246f Wladimir J. van der Laan: Merge #12793: qt: Avoid reseting on resetguisettigs=0...
< bitcoin-git> [bitcoin] laanwj closed pull request #12793: qt: Avoid reseting on resetguisettigs=0 (master...Mf1803-qtReset0) https://github.com/bitcoin/bitcoin/pull/12793
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/17b246f4b3f5...68484d64fd79
< bitcoin-git> bitcoin/master fc7c32f Gregory Sanders: do not truncate .dat extension for wallets in gui
< bitcoin-git> bitcoin/master 68484d6 Jonas Schnelli: Merge #12795: do not truncate .dat extension for wallets in gui...
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #12795: do not truncate .dat extension for wallets in gui (master...nowalletprune) https://github.com/bitcoin/bitcoin/pull/12795
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/68484d64fd79...3de01268b704
< bitcoin-git> bitcoin/master 1f45e21 practicalswift: scripted-diff: Convert 11 enums into scoped enums (C++11)...
< bitcoin-git> bitcoin/master 3de0126 Wladimir J. van der Laan: Merge #10742: scripted-diff: Use scoped enumerations (C++11, "enum class")...
< bitcoin-git> [bitcoin] laanwj closed pull request #10742: scripted-diff: Use scoped enumerations (C++11, "enum class") (master...scoped-enums) https://github.com/bitcoin/bitcoin/pull/10742
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3de01268b704...0415b1e6b2f7
< bitcoin-git> bitcoin/master 4ae7d15 MarcoFalke: init: Fix help message for checkblockindex
< bitcoin-git> bitcoin/master 0415b1e Wladimir J. van der Laan: Merge #12797: init: Fix help message for checkblockindex...
< bitcoin-git> [bitcoin] laanwj closed pull request #12797: init: Fix help message for checkblockindex (master...Mf1803-initHelpCheckBlockIndex) https://github.com/bitcoin/bitcoin/pull/12797
< bitcoin-git> [bitcoin] practicalswift opened pull request #12800: doc: Add note about our preference for scoped enumerations ("enum class") (master...document-enum-class-preference) https://github.com/bitcoin/bitcoin/pull/12800
< bitcoin-git> [bitcoin] practicalswift closed pull request #10638: rpc: Clarify pblock assumption via an assertion (master...nnonce) https://github.com/bitcoin/bitcoin/pull/10638
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0415b1e6b2f7...2d97611c41d9
< bitcoin-git> bitcoin/master 0fee2b4 practicalswift: doc: Add note about our preference for scoped enumerations ("enum class")
< bitcoin-git> bitcoin/master 2d97611 Wladimir J. van der Laan: Merge #12800: doc: Add note about our preference for scoped enumerations ("enum class")...
< bitcoin-git> [bitcoin] laanwj closed pull request #12800: doc: Add note about our preference for scoped enumerations ("enum class") (master...document-enum-class-preference) https://github.com/bitcoin/bitcoin/pull/12800
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2d97611c41d9...bf8fc7d352ad
< bitcoin-git> bitcoin/master d40f06a Pieter Wuille: Introduce interface for signing providers...
< bitcoin-git> bitcoin/master bf8fc7d Wladimir J. van der Laan: Merge #12714: Introduce interface for signing providers...
< bitcoin-git> [bitcoin] laanwj closed pull request #12714: Introduce interface for signing providers (master...201803_signingprovider) https://github.com/bitcoin/bitcoin/pull/12714
< arubi> sweet :)
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bf8fc7d352ad...e49ba2e23b78
< bitcoin-git> bitcoin/master 25eb9f5 João Barbosa: Inline CKeyStore::AddKey(const CKey &) in CBasicKeyStore
< bitcoin-git> bitcoin/master f381299 João Barbosa: Move CKeyStore::cs_KeyStore to CBasicKeyStore
< bitcoin-git> bitcoin/master e49ba2e Wladimir J. van der Laan: Merge #12762: Make CKeyStore an interface...
< bitcoin-git> [bitcoin] laanwj closed pull request #12762: Make CKeyStore an interface (master...2018-03-keystore) https://github.com/bitcoin/bitcoin/pull/12762
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e49ba2e23b78...b648974cc35a
< bitcoin-git> bitcoin/master 4d74c78 Gabriel Davidian: Add username and ip logging for RPC method requests
< bitcoin-git> bitcoin/master b648974 Wladimir J. van der Laan: Merge #12778: RPC: Add username and ip logging for RPC method requests...
< bitcoin-git> [bitcoin] laanwj closed pull request #12778: RPC: Add username and ip logging for RPC method requests (master...12223-rpc-log-username) https://github.com/bitcoin/bitcoin/pull/12778
< bitcoin-git> [bitcoin] laanwj closed pull request #12711: Update build-windows.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/12711
< everde> Hi, any news for d3 with X13
< everde> I did some changes on Antminer D3, some people is working with develop X13 algorithm
< bitcoin-git> [bitcoin] skeees opened pull request #12801: Add option to only notify after wallet transactions are confirmed (master...notifycount) https://github.com/bitcoin/bitcoin/pull/12801
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/b648974cc35a...534b8fa560a7
< bitcoin-git> bitcoin/master 386a6b6 Jonas Schnelli: Allow to optional specify the directory for the blocks storage
< bitcoin-git> bitcoin/master f38e4fd Jonas Schnelli: QA: Add -blocksdir test
< bitcoin-git> bitcoin/master a192636 Jonas Schnelli: -blocksdir: keep blockindex leveldb database in datadir
< bitcoin-git> [bitcoin] laanwj closed pull request #12653: Allow to optional specify the directory for the blocks storage (master...2018/03/blocksdir) https://github.com/bitcoin/bitcoin/pull/12653
< CryptoOdin> hi, any news respect D3 with x13 algorithm
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/534b8fa560a7...ac898b689c66
< bitcoin-git> bitcoin/master 1fdc7c4 Roman Zeyde: Make CTxMemPool::isSpent() const
< bitcoin-git> bitcoin/master 9cb9af8 Roman Zeyde: [REST] Handle UTXO retrieval when ignoring the mempool...
< bitcoin-git> bitcoin/master ac898b6 Wladimir J. van der Laan: Merge #12717: [REST] Handle UTXO retrieval when ignoring the mempool...
< bitcoin-git> [bitcoin] laanwj closed pull request #12717: [REST] Handle UTXO retrieval when ignoring the mempool (master...master) https://github.com/bitcoin/bitcoin/pull/12717
< bitcoin-git> [bitcoin] TheBlueMatt closed pull request #10692: Make mapBlockIndex and chainActive and all CBlockIndex*es const outside of validation/CChainState (master...2017-04-const-mapblockindex) https://github.com/bitcoin/bitcoin/pull/10692
< CryptoOdin> another advice
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #12802: Colonel Mustard in the Lounge with the Lead Pipe (killed mapBlockIndex outside of validation) (master...2018-03-const-cblockindex) https://github.com/bitcoin/bitcoin/pull/12802
< bitcoin-git> [bitcoin] sipa opened pull request #12803: Make BaseSignatureCreator a pure interface (master...201803_puresignaturecreator) https://github.com/bitcoin/bitcoin/pull/12803
< bitcoin-git> [bitcoin] jnewbery opened pull request #12804: [tests] Fix intermittent rpc_net.py failure. (master...rpc_net_py_fix) https://github.com/bitcoin/bitcoin/pull/12804
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #12806: qa: Fix function names in feature_blocksdir (master...Mf1803-qaBlocksdirFixup) https://github.com/bitcoin/bitcoin/pull/12806