< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/28485c783d82...6866b4912b80
< bitcoin-git> bitcoin/master 352d582 Marko Bencun: Add vConnect to CConnman::Options...
< bitcoin-git> bitcoin/master 6866b49 Wladimir J. van der Laan: Merge #10596: Add vConnect to CConnman::Options...
< bitcoin-git> [bitcoin] laanwj closed pull request #10596: Add vConnect to CConnman::Options (master...connmanoptions_connect) https://github.com/bitcoin/bitcoin/pull/10596
< bitcoin-git> [bitcoin] jtimon closed pull request #10822: RPC: Also serve txo from gettxout (not just utxo and mempool) (master...b15-rpc-txo) https://github.com/bitcoin/bitcoin/pull/10822
< esotericnonsense> a question which I think is related to core dev, apologies if not
< esotericnonsense> so the RPC interface should only be exposed to trusted clients in the sense of wallet commands etc
< esotericnonsense> however, is it otherwise expected to be hardened? e.g. if I have some software running in a VM or on another machine, and give it access to the RPC, would it be reasonable to assume that it shouldn't be able to raise privileges and execute commands on the host
< esotericnonsense> (basically, if i'm -disablewalleted, should I worry about anything other than possible DoS
< jonasschnelli> esotericnonsense: Even without your wallet enabled you should threat your RPC interface private.
< jonasschnelli> If you want to access it remotely, use VPN, SSH, stunnel or something.
< jonasschnelli> Don't expose the RPC interface to the public internet
< esotericnonsense> jonasschnelli: indeed. i suppose essentially what i'm wondering is how carefully I need to vet software which interfaces with the RPC.
< jonasschnelli> (you could use an Apache or so reverse proxy with SSL auth)
< jonasschnelli> I would do it carefully... use a VPN, reverse proxy or connect via SSH and use bitcoin-cli
< esotericnonsense> for example, core node running on host1, host2 runs some software which interfaces with host1, but host2 is malicious for whatever reason; can host1 be broken into (overflows on rpc commands, etc)
< esotericnonsense> i guess the actual answer is to run the core node in a sandbox to avoid this coming up at all
< jonasschnelli> if host2 is assumed to be malicious then it should not be capable to access to the RPC interface
< esotericnonsense> understood
< esotericnonsense> for reference i'm going to play around with writing some nodejs code to interface with the RPC and I find npm an utter clusterf.ck to deal with in terms of security :P
< bitcoin-git> [bitcoin] instagibbs opened pull request #11252: when clearing addrman clear mapInfo and mapAddr (master...recreateaddrman) https://github.com/bitcoin/bitcoin/pull/11252
< bitcoin-git> [bitcoin] JeremyRubin opened pull request #11253: Add != to Base58 class (master...add_neq_ctxdest) https://github.com/bitcoin/bitcoin/pull/11253
< cam9956> hello! anybody there?
< cam9956> i need some help
< cam9956> _flow_ are you there?
< meshcollider> Hi, what do you need help with?
< cam9956> i had a question, when i relese the coin does it have to be mined for anybody to get some or when i release it i get a few coins to sell out
< meshcollider> This channel is for discussion about bitcoin core development sorry, try #bitcoin or https://bitcoin.stackexchange.com/
< cam9956> yeah this is about bitcoin, i want to devalop my own coin with the bitcoin source
< meshcollider> Again, this is for bitcoin *core* development, not general questions sorry :)
< cam9956> what do you mean by "core development" and is there a irc i can contact for this?
< praxeology> cam9956: probably #bitcoin would be your best bet
< cam9956> ok
< praxeology> oh, your own coin
< cam9956> yeah with the bitcoin source code
< praxeology> go to #yet_another_bitcoin_fork
< cam9956> basically distributing
< cam9956> as the channel?
< Chicago> cam9956, They're saying go find another channel because clone coins are off-topic for here as they do not generally pertain to Bitcoin Core development.
< cam9956> what is bitcoin "core development" though
< meshcollider> Bitcoin Core is the reference client for bitcoin
< cam9956> oh ok bye!
< cam8857_> when i release the coin does it have to be mined for anybody to get some or when i release it i get a few coins to sell out?
< Chicago> cam8857_, you can have a 100% premine. Just let us know when you release so that we can buy them on the open market.
< Chicago> <.<
< Chicago> >.>
< cam8857_> premine? what do you mean?
< cam8857_> ohh wait im in the wrong irc...
< * kallewoof> pokes Travis with a stick.
< kallewoof> signraw works but sendraw fails in some cases.
< kallewoof> This function is supposed to find those cases and tell you without actually sending the tx to the network.
< dcousens> kallewoof: right, but the OP doesn't want to send at all, so that is OK?
< dcousens> signraw doesn't send either
< dcousens> ?
< kallewoof> Idea is, you sign, then you verify and if everything is cool you do whatever other last minute checks (double check addy or such) and then you finally sendraw.
< kallewoof> Or you mkae a tx using custom software and you just wanna see if it will relay without actually sending it.
< kallewoof> (signing using said custom sw)
< dcousens> kallewoof: right, but you can still use signrawtransaction in the custom software case
< dcousens> (I do)
< dcousens> I leave the private keys blank, and don't have a bitcoind w/ wallet support, and it verifies whether the tx is complete or has errors etc
< dcousens> kallewoof: eg I just created a tx with bitcoinjs, signed it using bitcoinjs, then signrawtx [txhex, inputs, []], complete: true, then I modified inputs to have a different id, same txhex, and I get a checksig error
< kallewoof> dcounsens: there are cases where signraw succeeds and sendraw fails, though. This is meant to catch those. Am I misundersatnding you?
< dcousens> kallewoof: right, signraw will fail for policy reasons, which is why I suggested, why not just a policy check flag?
< dcousens> rather than a new RPC command
< dcousens> uh, won't fail***
< dcousens> signraw won't fail for policy reasons
< sipa> signraw _only_ checks script validity
< sipa> not transaction validity
< sipa> things like locktime, feerate, inputs unspent, input value >= output value, ...
< sipa> that's not just policy
< dcousens> ok
< dcousens> ta :), i'll modify my reply
< dcousens> ta sipa
< dcousens> (and kallewoof )
< bitcoin-git> [bitcoin] JeremyRubin closed pull request #11253: Add != to Base58 class (master...add_neq_ctxdest) https://github.com/bitcoin/bitcoin/pull/11253
< promag> sipa: isn´t this https://github.com/bitcoin/bitcoin/pull/11117/files#r137145517 worth the change?
< bitcoin-git> [bitcoin] ysangkok opened pull request #11255: Release Notes: Fix SHA256-with-SSE4 PR link (0.15...patch-1) https://github.com/bitcoin/bitcoin/pull/11255
< bitcoin-git> [bitcoin] esotericnonsense opened pull request #11256: RPC: add weight to mempool entry output (master...2017-09-add-weight-to-mempool-entry) https://github.com/bitcoin/bitcoin/pull/11256
< MarcoFalke> promag: Great. Going to take a look today
< promag> MarcoFalke: review commit by commit, I think it's the best. I'll be around if you need to discuss it.
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to 0.15: https://github.com/bitcoin/bitcoin/compare/48790e0efaf4...134d824193a0
< bitcoin-git> bitcoin/0.15 58f1e54 Janus Troelsen: Release Notes: Fix SHA256-with-SSE4 PR link
< bitcoin-git> bitcoin/0.15 134d824 Wladimir J. van der Laan: Merge #11255: Release Notes: Fix SHA256-with-SSE4 PR link...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6866b4912b80...815fe62421cb
< bitcoin-git> bitcoin/master 0311836 Suhas Daftuar: Allow setting nMinimumChainWork on command line
< bitcoin-git> bitcoin/master eac64bb Suhas Daftuar: [qa] Test nMinimumChainWork...
< bitcoin-git> bitcoin/master 815fe62 Wladimir J. van der Laan: Merge #10357: Allow setting nMinimumChainWork on command line...
< bitcoin-git> [bitcoin] laanwj closed pull request #10357: Allow setting nMinimumChainWork on command line (master...2017-05-chainwork-commandline) https://github.com/bitcoin/bitcoin/pull/10357
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/815fe62421cb...5368614aa106
< bitcoin-git> bitcoin/master dea086f MeshCollider: Stop test_bitcoin-qt touching ~/.bitcoin
< bitcoin-git> bitcoin/master 5368614 Wladimir J. van der Laan: Merge #11210: Stop test_bitcoin-qt touching ~/.bitcoin...
< bitcoin-git> [bitcoin] laanwj closed pull request #11210: Stop test_bitcoin-qt touching ~/.bitcoin (master...201708_qt_test_dir) https://github.com/bitcoin/bitcoin/pull/11210
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5368614aa106...21e2f2f79961
< bitcoin-git> bitcoin/master 62ecce7 João Barbosa: [doc] Add RPC response notes
< bitcoin-git> bitcoin/master 21e2f2f Wladimir J. van der Laan: Merge #11135: Update developer notes with RPC response guidelines...
< bitcoin-git> [bitcoin] laanwj closed pull request #11135: Update developer notes with RPC response guidelines (master...2017-08-rpc-response-notes) https://github.com/bitcoin/bitcoin/pull/11135
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/21e2f2f79961...31809d6f8514
< bitcoin-git> bitcoin/master faa8d95 MarcoFalke: [qa] TestNode: Add wait_until_stopped helper method
< bitcoin-git> bitcoin/master 31809d6 MarcoFalke: Merge #11067: [qa] TestNode: Add wait_until_stopped helper method...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #11067: [qa] TestNode: Add wait_until_stopped helper method (master...Mf1708-qaTestnodeWaitStopHelper) https://github.com/bitcoin/bitcoin/pull/11067
< bitcoin-git> [bitcoin] jnewbery opened pull request #11258: [WIP] [rpc] Add initialblockdownload to getblockchaininfo (master...expose_ibd) https://github.com/bitcoin/bitcoin/pull/11258
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/31809d6f8514...59e17899a762
< bitcoin-git> bitcoin/master 37495e0 Jeremy Rubin: Reorder C{,Mutable}Transaction for better packing
< bitcoin-git> bitcoin/master 59e1789 Wladimir J. van der Laan: Merge #8330: Structure Packing Optimizations in C{,Mutable}Transaction...
< GAit> is github user promag in IRC?
< GAit> oh he usually is just not now, fair enough
< sipa> sometimes
< sipa> he's here now :p
< promag> :)
< promag> ´
< promag> sup?
< promag> GAit: need something?
< GAit> oh I just wanted to say I am not ignoring your suggestion re: 11099 with the multiple failure modes of DumpMempool, i.e. the test doesn't exercise the "return false" within the exception handler but i think that's much harder to cause and i wanted to minimize changes to DumpMempool or how it affects other callers
< GAit> and also i wanted to thank you for all the feedback, very valuable
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/59e17899a762...d745b4cf7b09
< bitcoin-git> bitcoin/master 7e5d596 Suhas Daftuar: RPC: add wtxid to mempool entry output
< bitcoin-git> bitcoin/master 617c459 Suhas Daftuar: qa: rpc test for wtxid in mempool entry
< bitcoin-git> bitcoin/master d745b4c Wladimir J. van der Laan: Merge #11203: rpc: add wtxid to mempool entry output...
< promag> there's also "Also, RenameOver() doesn't throw, and IMO we should fail if the mempool.dat can't be updated"
< promag> maybe in a follow up patch add handling for that
< GAit> right but that's IMHO for a different PR
< bitcoin-git> [bitcoin] laanwj closed pull request #11203: rpc: add wtxid to mempool entry output (master...2017-08-add-wtxid-to-mempool-entry) https://github.com/bitcoin/bitcoin/pull/11203
< promag> voila
< promag> I can create that, it's on the todo list
< GAit> i'll be happy to review
< bitcoin-git> [bitcoin] laanwj closed pull request #10872: Docs: Syntax highlight shell commands in the building notes (master...hilite) https://github.com/bitcoin/bitcoin/pull/10872
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d745b4cf7b09...39ae41389a86
< bitcoin-git> bitcoin/master 41bf159 practicalswift: Remove unreachable code
< bitcoin-git> bitcoin/master 39ae413 Wladimir J. van der Laan: Merge #10845: Remove unreachable code...
< bitcoin-git> [bitcoin] laanwj closed pull request #10845: Remove unreachable code (master...remove-unreachable-code) https://github.com/bitcoin/bitcoin/pull/10845
< promag> jnewbery: in portuguese, xpto it's like foo :)
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/39ae41389a86...961901f77e55
< bitcoin-git> bitcoin/master 5c8ff0d Pieter Wuille: Introduce wrappers around CBitcoinAddress...
< bitcoin-git> bitcoin/master 864cd27 Pieter Wuille: Move CBitcoinAddress to base58.cpp
< bitcoin-git> bitcoin/master 961901f Wladimir J. van der Laan: Merge #11117: Prepare for non-Base58 addresses...
< bitcoin-git> [bitcoin] laanwj closed pull request #11117: Prepare for non-Base58 addresses (master...201708_nocbitcoinaddress) https://github.com/bitcoin/bitcoin/pull/11117
< bitcoin-git> [bitcoin] promag opened pull request #11259: Remove duplicate destination decoding (master...2017-09-remove-duplicate-destination-decoding) https://github.com/bitcoin/bitcoin/pull/11259
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/961901f77e55...bc561b4b7d6a
< bitcoin-git> bitcoin/master 467cbbc Lawrence Nahum: Add return value to DumpMempool
< bitcoin-git> bitcoin/master 1aa97ee Lawrence Nahum: Add savemempool RPC
< bitcoin-git> bitcoin/master bc561b4 MarcoFalke: Merge #11099: [RPC][mempool]: Add savemempool RPC...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #11099: [RPC][mempool]: Add savemempool RPC (master...dump_mempool_rpc) https://github.com/bitcoin/bitcoin/pull/11099
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bc561b4b7d6a...6acdb1fab770
< bitcoin-git> bitcoin/master c001992 MeshCollider: Fix potential null dereferences
< bitcoin-git> bitcoin/master 6acdb1f Wladimir J. van der Laan: Merge #11238: Add assertions before potential null deferences...
< bitcoin-git> [bitcoin] laanwj closed pull request #11238: Add assertions before potential null deferences (master...201708_fbinfer_fixes) https://github.com/bitcoin/bitcoin/pull/11238
< jnewbery> promag : obrigado :)
< promag> jnewbery: np
< bitcoin-git> [bitcoin] laanwj closed pull request #10889: Docs: Run windows build in a clean path (master...docswin) https://github.com/bitcoin/bitcoin/pull/10889
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #11260: travis: Check that default datadir is never created, Check SHAs only once (master...Mf1708-travisYaml) https://github.com/bitcoin/bitcoin/pull/11260
< wumpus> looks like we have a few new warnings:
< wumpus> base58.cpp:308:6: warning: ‘bool {anonymous}::CBitcoinAddress::IsScript() const’ defined but not used [-Wunused-function]
< wumpus> base58.cpp:298:6: warning: ‘bool {anonymous}::CBitcoinAddress::GetKeyID(CKeyID&) const’ defined but not used [-Wunused-function]
< wumpus> promag: mind to addresss ^^ in #11259?
< promag> np
< wumpus> at least, if it's not result of a bug or something, haven't looked in detail yet
< promag> I'll take a look
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6acdb1fab770...bcc8a620eae9
< bitcoin-git> bitcoin/master 478d4fb Sjors Provoost: [docs] explain how to recompile only what bitcoind tests need...
< bitcoin-git> bitcoin/master bcc8a62 Wladimir J. van der Laan: Merge #11219: [docs] explain how to recompile a modified unit test...
< bitcoin-git> [bitcoin] laanwj closed pull request #11219: [docs] explain how to recompile a modified unit test (master...patch-1) https://github.com/bitcoin/bitcoin/pull/11219
< wumpus> so the compiler can identify unused *methods* now? that's interesting - i guess because it's local to the file
< wumpus> that was with gcc 6.3.0, btw
< promag> wumpus: rebased, new commit
< promag> wumpus: I think so too
< wumpus> promag: thanks, will rebuild to check
< bitcoin-git> [bitcoin] practicalswift opened pull request #11261: scripted-diff: Use <cxxx> instead of deprecated <xxx.h> when including C compatibility headers (master...cinclude) https://github.com/bitcoin/bitcoin/pull/11261
< promag> is it me or github mail notifications are lagging?
< wumpus> yes, it seems slow
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bcc8a620eae9...089b742a210b
< bitcoin-git> bitcoin/master 8d0041e João Barbosa: Remove unused GetKeyID and IsScript methods from CBitcoinAddress
< bitcoin-git> bitcoin/master 86e6dd4 João Barbosa: Remove duplicate destination decoding
< bitcoin-git> bitcoin/master 089b742 Wladimir J. van der Laan: Merge #11259: Remove duplicate destination decoding...
< bitcoin-git> [bitcoin] laanwj closed pull request #11259: Remove duplicate destination decoding (master...2017-09-remove-duplicate-destination-decoding) https://github.com/bitcoin/bitcoin/pull/11259
< MarcoFalke> wumpus: 10838
< wumpus> MarcoFalke: hmm?
< MarcoFalke> ready for merge ;)
< promag> MarcoFalke: 11125
< wumpus> even without your release notes change? ok
< wumpus> yes 11125 is ready
< MarcoFalke> The release notes change will force another pull to be rebased
< MarcoFalke> , which already has some utACKs
< wumpus> true
< wumpus> right, I missed that there was a reasoning behind skipping it
< promag> sipa: care to rebase 11167?
< promag> I don't mind adding the release notes..
< MarcoFalke> wumpus: I never shared the reasoning. Either BlueMatt missed it or did it on purpose. Either way is fine, heh
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/089b742a210b...66a5b419eff5
< bitcoin-git> bitcoin/master aece8a4 Matt Corallo: (finally) remove getinfo in favor of more module-specific infos
< bitcoin-git> bitcoin/master 66a5b41 Wladimir J. van der Laan: Merge #10838: (finally) remove getinfo...
< bitcoin-git> [bitcoin] laanwj closed pull request #10838: (finally) remove getinfo (master...2017-07-seriously-fuck-getinfo) https://github.com/bitcoin/bitcoin/pull/10838
< wumpus> argh 11125 needs rebase too, now @promag
< BlueMatt> hmmm, what'd i do?
< wumpus> conflict: test/functional/bitcoin_cli.py
< promag> great branch name 2017-07-seriously-fuck-getinfo
< wumpus> nothing that could be avoided anyhwo
< promag> wumpus: 1 sec
< wumpus> lol nice
< promag> wumpus: rebased 11125
< wumpus> thanks
< promag> renamed cli_get_info to cli_response and rpc_get_info to rpc_response, I can revert thou
< promag> I was going to inline like assert_equal(self.nodes[0].cli.getblockchaininfo(), self.nodes[0].getblockchaininfo())
< meshcollider> how is fanquake able to add and remove labels, etc if hes not a member?
< meshcollider> or is he a member but just hasn't set it to display
< wumpus> he is a member
< bitcoin-git> [bitcoin] laanwj pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/66a5b419eff5...645a7ecc0b8d
< bitcoin-git> bitcoin/master 7696841 João Barbosa: Fix style in -stdin and -stdinrpcpass handling
< bitcoin-git> bitcoin/master e127494 João Barbosa: [test] Improve assert_raises_jsonrpc docstring
< bitcoin-git> bitcoin/master 5c18a84 João Barbosa: [test] Add support for custom arguments to TestNodeCLI
< bitcoin-git> [bitcoin] laanwj closed pull request #11125: Add bitcoin-cli -stdin and -stdinrpcpass functional tests (master...2017-08-stdinrpcpass-functional-test) https://github.com/bitcoin/bitcoin/pull/11125
< meshcollider> things have been moving so fast these last couple of days, its awesome
< meshcollider> so many merges
< wumpus> yes we've caught up a little bit on the PR backlog :)
< MarcoFalke> Like 12 new pulls and 14 merged/closed. Delta is like two or something
< wumpus> yeah...
< promag> still, merge fest!
< sipa> ya
< MarcoFalke> ja, who's not ready for bed by now?
< meshcollider> its only midday over here in NZ ;)
< promag> 1am here, must take kids to school by 8am, I can troll a bit more