1 2017-09-26T00:00:43  *** Ylbam has quit IRC
  2 2017-09-26T00:21:46  *** Miezel has quit IRC
  3 2017-09-26T00:27:13  *** chjj has joined #bitcoin-core-dev
  4 2017-09-26T00:58:03  *** dabura667 has joined #bitcoin-core-dev
  5 2017-09-26T00:59:05  *** promag has joined #bitcoin-core-dev
  6 2017-09-26T00:59:50  *** Miezel has joined #bitcoin-core-dev
  7 2017-09-26T01:08:28  *** Alina-malina has quit IRC
  8 2017-09-26T01:09:50  *** promag has quit IRC
  9 2017-09-26T01:10:02  *** Miezel has quit IRC
 10 2017-09-26T01:10:25  *** Alina-malina has joined #bitcoin-core-dev
 11 2017-09-26T01:11:06  *** Miezel has joined #bitcoin-core-dev
 12 2017-09-26T01:15:03  *** dynm_ has joined #bitcoin-core-dev
 13 2017-09-26T01:18:06  *** [Author] has joined #bitcoin-core-dev
 14 2017-09-26T01:20:27  *** promag has joined #bitcoin-core-dev
 15 2017-09-26T01:37:34  *** promag has quit IRC
 16 2017-09-26T01:49:56  <bitcoin-git> [bitcoin] sipa opened pull request #11403: SegWit wallet support (master...201709_segwitwallet2) https://github.com/bitcoin/bitcoin/pull/11403
 17 2017-09-26T01:53:43  *** merehap has quit IRC
 18 2017-09-26T01:53:55  *** merehap has joined #bitcoin-core-dev
 19 2017-09-26T01:55:27  *** RubenSomsen has joined #bitcoin-core-dev
 20 2017-09-26T01:55:31  <analiser> look like #10783 041dad9 RCP Various rpc argument fixes(instagiibbs) in the 0.15 channelog is what cause some errors in python bicoinlib module i just solve the issue removing from module bitcoin SelectParams() attribute and from bitcoin.rpc remove Proxy() attribute hope that work for any here
 21 2017-09-26T01:55:33  <gribble> https://github.com/bitcoin/bitcoin/issues/10783 | [RPC] Various rpc argument fixes by instagibbs · Pull Request #10783 · bitcoin/bitcoin · GitHub
 22 2017-09-26T02:00:08  *** merehap has quit IRC
 23 2017-09-26T02:00:20  *** merehap has joined #bitcoin-core-dev
 24 2017-09-26T02:01:46  *** merehap_ has joined #bitcoin-core-dev
 25 2017-09-26T02:02:25  *** merehap has quit IRC
 26 2017-09-26T02:21:29  *** Miezel has quit IRC
 27 2017-09-26T02:46:34  *** Murch has joined #bitcoin-core-dev
 28 2017-09-26T02:59:00  <bitcoin-git> [bitcoin] jtimon opened pull request #11404: RPC: Chainparams: Simplify -rpcport and CBaseMainParams  (master...b16-chainparams-rpcport) https://github.com/bitcoin/bitcoin/pull/11404
 29 2017-09-26T03:20:24  *** ensign has quit IRC
 30 2017-09-26T03:24:13  *** ensign has joined #bitcoin-core-dev
 31 2017-09-26T03:24:42  *** merehap_ has quit IRC
 32 2017-09-26T03:24:52  *** merehap_ has joined #bitcoin-core-dev
 33 2017-09-26T03:26:22  *** merehap_ has quit IRC
 34 2017-09-26T03:26:32  *** merehap_ has joined #bitcoin-core-dev
 35 2017-09-26T03:37:43  *** jcorgan has quit IRC
 36 2017-09-26T03:40:05  *** RubenSomsen has quit IRC
 37 2017-09-26T03:42:50  *** ula has quit IRC
 38 2017-09-26T03:45:32  *** Murch has quit IRC
 39 2017-09-26T03:46:07  *** tknp has joined #bitcoin-core-dev
 40 2017-09-26T03:46:13  *** RubenSomsen has joined #bitcoin-core-dev
 41 2017-09-26T03:53:53  *** dynm_ has quit IRC
 42 2017-09-26T03:54:07  *** Ruben has joined #bitcoin-core-dev
 43 2017-09-26T03:56:32  *** RubenSomsen has quit IRC
 44 2017-09-26T03:58:35  *** Ruben has quit IRC
 45 2017-09-26T04:00:08  *** PatBoy has quit IRC
 46 2017-09-26T04:08:34  *** chartractegg has joined #bitcoin-core-dev
 47 2017-09-26T04:15:41  *** instagibbs has quit IRC
 48 2017-09-26T04:16:22  *** chartractegg has quit IRC
 49 2017-09-26T04:24:25  *** instagibbs has joined #bitcoin-core-dev
 50 2017-09-26T04:28:43  *** wittysense has quit IRC
 51 2017-09-26T04:41:01  *** ThomasV has joined #bitcoin-core-dev
 52 2017-09-26T04:41:46  *** justanotheruser has quit IRC
 53 2017-09-26T04:50:00  *** ThomasV has quit IRC
 54 2017-09-26T05:02:29  *** justanotheruser has joined #bitcoin-core-dev
 55 2017-09-26T05:03:06  *** tripleslash has quit IRC
 56 2017-09-26T05:12:58  *** AaronvanW has joined #bitcoin-core-dev
 57 2017-09-26T05:19:28  *** chjj has quit IRC
 58 2017-09-26T05:23:21  *** AaronvanW has quit IRC
 59 2017-09-26T05:25:12  *** andytoshi has quit IRC
 60 2017-09-26T05:25:46  *** andytoshi has joined #bitcoin-core-dev
 61 2017-09-26T05:26:57  *** instagibbs has quit IRC
 62 2017-09-26T05:29:00  *** instagibbs has joined #bitcoin-core-dev
 63 2017-09-26T05:44:24  *** Murch has joined #bitcoin-core-dev
 64 2017-09-26T05:53:50  *** tknp has quit IRC
 65 2017-09-26T05:57:31  *** Alina-malina has quit IRC
 66 2017-09-26T05:57:31  *** Alina-malina has joined #bitcoin-core-dev
 67 2017-09-26T06:00:16  *** Murch has quit IRC
 68 2017-09-26T06:03:38  *** Murch has joined #bitcoin-core-dev
 69 2017-09-26T06:07:06  *** Murch has quit IRC
 70 2017-09-26T06:11:16  *** meshcollider has quit IRC
 71 2017-09-26T06:16:38  *** RubenSomsen has joined #bitcoin-core-dev
 72 2017-09-26T06:24:12  *** arubi has quit IRC
 73 2017-09-26T06:32:37  *** JackH has quit IRC
 74 2017-09-26T06:41:50  *** arubi has joined #bitcoin-core-dev
 75 2017-09-26T06:42:36  *** Deacyded has joined #bitcoin-core-dev
 76 2017-09-26T06:46:15  *** Deacydal has quit IRC
 77 2017-09-26T06:46:18  *** dermoth has joined #bitcoin-core-dev
 78 2017-09-26T06:49:05  *** BashCo has quit IRC
 79 2017-09-26T06:49:46  *** BashCo has joined #bitcoin-core-dev
 80 2017-09-26T06:52:17  *** EVM has joined #bitcoin-core-dev
 81 2017-09-26T06:52:25  <EVM> hi
 82 2017-09-26T06:54:13  *** BashCo has quit IRC
 83 2017-09-26T06:59:37  *** JackH has joined #bitcoin-core-dev
 84 2017-09-26T07:01:45  *** meshcollider has joined #bitcoin-core-dev
 85 2017-09-26T07:09:56  *** arubi has quit IRC
 86 2017-09-26T07:10:22  *** arubi has joined #bitcoin-core-dev
 87 2017-09-26T07:10:57  *** BashCo has joined #bitcoin-core-dev
 88 2017-09-26T07:14:20  *** bigcheap has joined #bitcoin-core-dev
 89 2017-09-26T07:22:05  *** arubi has quit IRC
 90 2017-09-26T07:22:37  *** arubi has joined #bitcoin-core-dev
 91 2017-09-26T07:34:33  *** circ-user-2HbxE has joined #bitcoin-core-dev
 92 2017-09-26T07:38:17  *** promag has joined #bitcoin-core-dev
 93 2017-09-26T07:41:16  *** promag has quit IRC
 94 2017-09-26T07:46:13  *** promag has joined #bitcoin-core-dev
 95 2017-09-26T07:50:34  *** promag has quit IRC
 96 2017-09-26T07:58:05  *** timothy has joined #bitcoin-core-dev
 97 2017-09-26T07:59:08  *** afk11 has quit IRC
 98 2017-09-26T08:00:03  *** afk11 has joined #bitcoin-core-dev
 99 2017-09-26T08:02:30  *** EVM has quit IRC
100 2017-09-26T08:02:59  *** pbase has joined #bitcoin-core-dev
101 2017-09-26T08:12:59  *** Victor_sueca has joined #bitcoin-core-dev
102 2017-09-26T08:16:07  *** Victorsueca has quit IRC
103 2017-09-26T08:16:17  *** promag has joined #bitcoin-core-dev
104 2017-09-26T08:27:51  *** goatpig has quit IRC
105 2017-09-26T08:55:32  *** ula has joined #bitcoin-core-dev
106 2017-09-26T08:56:16  *** pbase has quit IRC
107 2017-09-26T09:09:18  *** promag has quit IRC
108 2017-09-26T09:31:05  *** Guyver2 has joined #bitcoin-core-dev
109 2017-09-26T09:48:13  *** Miezel has joined #bitcoin-core-dev
110 2017-09-26T09:51:00  *** bigcheap has quit IRC
111 2017-09-26T10:01:32  *** dabura667 has quit IRC
112 2017-09-26T10:28:16  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8cf88b4aaeaf...67879b7c4431
113 2017-09-26T10:28:16  <bitcoin-git> bitcoin/master 28d4542 Matt Corallo: Disallow uncompressed pubkeys in bitcoin-tx [multisig] output adds
114 2017-09-26T10:28:17  <bitcoin-git> bitcoin/master 67879b7 Wladimir J. van der Laan: Merge #11377: Disallow uncompressed pubkeys in bitcoin-tx [multisig] output adds...
115 2017-09-26T10:28:51  <bitcoin-git> [bitcoin] laanwj closed pull request #11377: Disallow uncompressed pubkeys in bitcoin-tx [multisig] output adds (master...2017-09-bitcoin-tx-uncompressed-segwit) https://github.com/bitcoin/bitcoin/pull/11377
116 2017-09-26T10:29:10  *** circ-user-2HbxE has quit IRC
117 2017-09-26T10:29:26  *** circ-user-2HbxE has joined #bitcoin-core-dev
118 2017-09-26T11:08:23  *** circ-user-2HbxE has quit IRC
119 2017-09-26T11:13:08  *** justanotheruser has quit IRC
120 2017-09-26T11:42:28  *** promag has joined #bitcoin-core-dev
121 2017-09-26T11:56:27  *** circ-user-2HbxE has joined #bitcoin-core-dev
122 2017-09-26T12:08:09  *** PatBoy has joined #bitcoin-core-dev
123 2017-09-26T12:09:54  *** PatBoy has joined #bitcoin-core-dev
124 2017-09-26T12:25:13  *** Geoffy has joined #bitcoin-core-dev
125 2017-09-26T12:29:24  *** circ-user-2HbxE has quit IRC
126 2017-09-26T12:29:51  *** wittysense has joined #bitcoin-core-dev
127 2017-09-26T12:36:34  *** SopaXorzTaker has joined #bitcoin-core-dev
128 2017-09-26T12:39:44  *** promag has quit IRC
129 2017-09-26T12:43:19  *** AaronvanW has joined #bitcoin-core-dev
130 2017-09-26T12:47:35  *** LeMiner2 has joined #bitcoin-core-dev
131 2017-09-26T12:49:36  *** LeMiner has quit IRC
132 2017-09-26T12:49:36  *** LeMiner2 is now known as LeMiner
133 2017-09-26T12:53:47  *** Victor_sueca is now known as Victorsueca
134 2017-09-26T12:54:10  *** Chris_Stewart_5 has joined #bitcoin-core-dev
135 2017-09-26T13:00:53  *** justanotheruser has joined #bitcoin-core-dev
136 2017-09-26T13:02:30  *** promag has joined #bitcoin-core-dev
137 2017-09-26T13:04:37  *** AaronvanW has quit IRC
138 2017-09-26T13:06:40  *** promag has quit IRC
139 2017-09-26T13:08:54  *** justanotheruser has quit IRC
140 2017-09-26T13:09:14  *** justanotheruser has joined #bitcoin-core-dev
141 2017-09-26T13:12:33  *** promag has joined #bitcoin-core-dev
142 2017-09-26T13:15:26  *** AaronvanW has joined #bitcoin-core-dev
143 2017-09-26T13:28:52  *** Evel-Knievel has quit IRC
144 2017-09-26T13:32:17  *** Chris_Stewart_5 has quit IRC
145 2017-09-26T13:37:01  *** SopaXorzTaker has quit IRC
146 2017-09-26T13:38:17  *** SopaXorzTaker has joined #bitcoin-core-dev
147 2017-09-26T13:39:29  *** promag has quit IRC
148 2017-09-26T13:46:31  *** promag has joined #bitcoin-core-dev
149 2017-09-26T13:48:11  *** Chris_Stewart_5 has joined #bitcoin-core-dev
150 2017-09-26T13:50:15  *** SopaXorzTaker has quit IRC
151 2017-09-26T13:50:43  *** promag has quit IRC
152 2017-09-26T13:59:15  *** SopaXorzTaker has joined #bitcoin-core-dev
153 2017-09-26T14:01:34  *** justanotheruser has quit IRC
154 2017-09-26T14:06:02  *** AaronvanW has quit IRC
155 2017-09-26T14:11:16  *** meshcollider has quit IRC
156 2017-09-26T14:12:02  *** d9b4bef9 has quit IRC
157 2017-09-26T14:15:07  *** d9b4bef9 has joined #bitcoin-core-dev
158 2017-09-26T14:15:08  *** dermoth has quit IRC
159 2017-09-26T14:17:19  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/67879b7c4431...0f8e09599db6
160 2017-09-26T14:17:19  <bitcoin-git> bitcoin/master df10edf MeshCollider: More user-friendly error message when partially signing
161 2017-09-26T14:17:20  <bitcoin-git> bitcoin/master 0f8e095 Wladimir J. van der Laan: Merge #11288: More user-friendly error message when partially signing...
162 2017-09-26T14:17:23  *** dermoth has joined #bitcoin-core-dev
163 2017-09-26T14:18:15  <bitcoin-git> [bitcoin] laanwj closed pull request #11288: More user-friendly error message when partially signing (master...201709_partial_sign_error) https://github.com/bitcoin/bitcoin/pull/11288
164 2017-09-26T14:21:04  *** victorSN has joined #bitcoin-core-dev
165 2017-09-26T14:21:10  *** promag has joined #bitcoin-core-dev
166 2017-09-26T14:24:54  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0f8e09599db6...dc597bb895ef
167 2017-09-26T14:24:54  <bitcoin-git> bitcoin/master d01a968 Suhas Daftuar: wallet: update stored witness in AddToWallet...
168 2017-09-26T14:24:55  <bitcoin-git> bitcoin/master dc597bb Wladimir J. van der Laan: Merge #11225: wallet: update stored witness in AddToWallet...
169 2017-09-26T14:25:34  <bitcoin-git> [bitcoin] laanwj closed pull request #11225: wallet: update stored witness in AddToWallet (master...2017-09-update-tx-witness) https://github.com/bitcoin/bitcoin/pull/11225
170 2017-09-26T14:32:58  *** promag has quit IRC
171 2017-09-26T14:34:42  *** Chris_Stewart_5 has quit IRC
172 2017-09-26T14:38:01  *** AaronvanW has joined #bitcoin-core-dev
173 2017-09-26T14:49:15  *** Chris_Stewart_5 has joined #bitcoin-core-dev
174 2017-09-26T14:51:00  *** AaronvanW has quit IRC
175 2017-09-26T14:56:10  *** RubenSomsen has quit IRC
176 2017-09-26T15:16:33  *** AaronvanW has joined #bitcoin-core-dev
177 2017-09-26T15:43:15  *** Miezel has quit IRC
178 2017-09-26T15:44:41  *** Miezel has joined #bitcoin-core-dev
179 2017-09-26T15:52:37  *** promag has joined #bitcoin-core-dev
180 2017-09-26T15:54:22  *** BashCo has quit IRC
181 2017-09-26T15:57:06  *** promag has quit IRC
182 2017-09-26T15:57:54  *** RubenSomsen has joined #bitcoin-core-dev
183 2017-09-26T15:58:37  *** promag has joined #bitcoin-core-dev
184 2017-09-26T16:03:58  <promag> jnewbery: https://github.com/bitcoin/bitcoin/pull/11031/commits/a6919d9a6908efaf0c913cccda21d63ed1517b3c s/enablerpcmethod/deprecatedrpc
185 2017-09-26T16:05:39  *** promag has quit IRC
186 2017-09-26T16:06:05  *** benma has joined #bitcoin-core-dev
187 2017-09-26T16:09:41  *** Miezel has quit IRC
188 2017-09-26T16:11:22  *** Miezel has joined #bitcoin-core-dev
189 2017-09-26T16:19:57  *** BashCo has joined #bitcoin-core-dev
190 2017-09-26T16:27:26  <jnewbery> promag: thanks, fixed
191 2017-09-26T16:43:42  *** Ylbam has joined #bitcoin-core-dev
192 2017-09-26T16:52:12  <maaku> I'm considering removing the 200 non-push opcode limit and SIGOP counting for my tail-call proposal. I would be interested in feedback as to why this would be a terrible idea.
193 2017-09-26T16:54:18  *** AaronvanW has quit IRC
194 2017-09-26T16:58:58  *** AaronvanW has joined #bitcoin-core-dev
195 2017-09-26T17:02:17  *** AaronvanW has quit IRC
196 2017-09-26T17:05:25  *** AaronvanW has joined #bitcoin-core-dev
197 2017-09-26T17:05:58  *** victorSN has quit IRC
198 2017-09-26T17:06:05  *** Aaronvan_ has joined #bitcoin-core-dev
199 2017-09-26T17:10:11  *** AaronvanW has quit IRC
200 2017-09-26T17:11:33  *** Aaronvan_ has quit IRC
201 2017-09-26T17:28:28  *** jtimon has joined #bitcoin-core-dev
202 2017-09-26T17:28:58  *** goatpig has joined #bitcoin-core-dev
203 2017-09-26T17:30:12  *** benma has quit IRC
204 2017-09-26T17:30:51  *** Chris_Stewart_5 has quit IRC
205 2017-09-26T17:33:35  *** AaronvanW has joined #bitcoin-core-dev
206 2017-09-26T17:36:26  <bitcoin-git> [bitcoin] jtimon closed pull request #11404: RPC: Chainparams: Simplify -rpcport and CBaseMainParams  (master...b16-chainparams-rpcport) https://github.com/bitcoin/bitcoin/pull/11404
207 2017-09-26T17:36:45  *** Murch has joined #bitcoin-core-dev
208 2017-09-26T17:43:23  *** Chris_Stewart_5 has joined #bitcoin-core-dev
209 2017-09-26T17:44:21  *** PaulCapestany has quit IRC
210 2017-09-26T17:49:47  *** PaulCapestany has joined #bitcoin-core-dev
211 2017-09-26T17:51:49  <instagibbs> what interval if any does travis run extended functional tests?
212 2017-09-26T17:53:46  *** AaronvanW has quit IRC
213 2017-09-26T17:57:49  *** AaronvanW has joined #bitcoin-core-dev
214 2017-09-26T18:11:44  *** analiser has quit IRC
215 2017-09-26T18:18:19  *** Squidicuz has quit IRC
216 2017-09-26T18:22:35  *** victorSN has joined #bitcoin-core-dev
217 2017-09-26T18:24:04  *** victorSN has quit IRC
218 2017-09-26T18:31:28  *** Chris_Stewart_5 has quit IRC
219 2017-09-26T18:32:01  <jtimon> sipa: what's the reasoning for the bias for nodes using the default port? that seems weird
220 2017-09-26T18:33:03  <sipa> jtimon: otherwise you can spam the network with addr messages for services you don't like, and get the whole bitcoin network to try connecting to them
221 2017-09-26T18:33:39  <jtimon> how is that affected by the port, I don't follow...
222 2017-09-26T18:33:55  <sipa> with the bias, that can pretty much only happen for P2P default port, which is very unlikely to have any interesting services running, other than bitcoin :)
223 2017-09-26T18:34:29  <jtimon> oh, I see
224 2017-09-26T18:34:44  <jtimon> I think I get it...
225 2017-09-26T18:34:47  *** victorSN has joined #bitcoin-core-dev
226 2017-09-26T18:35:19  <jtimon> in any case, as said seems unrelated to #11404
227 2017-09-26T18:35:30  <gribble> https://github.com/bitcoin/bitcoin/issues/11404 | RPC: Chainparams: Simplify -rpcport and CBaseMainParams by jtimon · Pull Request #11404 · bitcoin/bitcoin · GitHub
228 2017-09-26T18:35:49  <sipa> well you're giving all networks the same default port
229 2017-09-26T18:36:04  <sipa> meaning that if you want to run more than one network on the same machine, one of them will need a non-default port
230 2017-09-26T18:36:42  <jtimon> the same defaul rpc port
231 2017-09-26T18:36:55  <esotericnonsense> yes, it's the rpc port only in the PR, was a bit confused by your comment sipa
232 2017-09-26T18:37:02  <sipa> oh!
233 2017-09-26T18:37:08  <sipa> this is about the RPC port
234 2017-09-26T18:37:11  <esotericnonsense> i still dislike the behaviour of having to specify a different rpcport for testnet/regtest though
235 2017-09-26T18:37:22  <esotericnonsense> though that's just a quality-of-life annoyance
236 2017-09-26T18:37:31  <sipa> yes, agree - though that's much less bad than having the same P2P port
237 2017-09-26T18:37:54  <sipa> sorry, i misunderstood!
238 2017-09-26T18:38:47  <esotericnonsense> while we're on the subject is there any preference for connecting to nodes with a different IP (perhaps based on this standard port bias)? e.g. if I run 50000 nodes from one IP and spam addr messages
239 2017-09-26T18:38:57  <gmaxwell> it just makes them unusable by default, I cannot fathom why we would do that.
240 2017-09-26T18:39:53  <sipa> esotericnonsense: we never make more than 1 outgoing connection to the same 'network group' (whose definition is dependent on the network, but for IPv4 it's a /16 i think)
241 2017-09-26T18:40:02  <esotericnonsense> aha.
242 2017-09-26T18:40:41  <sipa> and without -addnode we'll never connect to an unroutable address (so not to 192.168.x, not, not 10.x, etc)
243 2017-09-26T18:41:15  <sipa> i think we should at some point have local network discovery and try to connect within your own network, but not using the same external connection slots)
244 2017-09-26T18:42:30  <gmaxwell> sipa: I started working on that and didn't have a good way to eliminate the duplicate connections.
245 2017-09-26T18:42:43  <gmaxwell> E.g. hosts A and B on the local network, both connect to each other.
246 2017-09-26T18:43:53  <gmaxwell> I thought perhaps just a stupid hurestic, "if you listen, connect only to lower sort-order addresses than yours".
247 2017-09-26T18:44:43  <sipa> or just test for duplicate IP pairs? within a local network you can probably assume that both parties will see each other's by their actual IP (not NAT etc0
248 2017-09-26T18:45:19  <sipa> so don't connect out to any IP you're seeing an incoming connection from
249 2017-09-26T18:46:29  <gmaxwell> Good point. What about the race when two connect at the same time
250 2017-09-26T18:46:44  <jtimon> esotericnonsense: for bitcoin-cli, specifying -testnet does the same as -rpcport=18332, if you specify rpcport, specifying -testnet is completely meaningless
251 2017-09-26T18:47:03  *** Chris_Stewart_5 has joined #bitcoin-core-dev
252 2017-09-26T18:47:05  *** AaronvanW has quit IRC
253 2017-09-26T18:47:38  <jtimon> gmaxwell: what is made unusable by default? I don't follow
254 2017-09-26T18:48:16  <sipa> gmaxwell: if you have a duplicate pair among actual connections, and your IP is the lexicographically lower of the two, disconnect?
255 2017-09-26T18:48:54  <Sentineo> gmaxwell: sipa it sounds like split horizont in routing protocols what discuss ...
256 2017-09-26T18:49:07  <Sentineo> what you discuss
257 2017-09-26T18:49:29  <gmaxwell> jtimon: Making everything use the same rpc port means you cannot run testnet and bitcoin at the same time anymore except through manually configuring them.
258 2017-09-26T18:50:29  <Sentineo> btw sipa why is a node keeping 8 connetions outside ? I have a node with 123 connection, 8 of them always go outside. I would assume my node should prefer inbound connections.
259 2017-09-26T18:50:48  <jtimon> oh, that. yeah. if you want to use both in the same machine you have to specify a different rpc port in one of the,
260 2017-09-26T18:50:58  <jtimon> them
261 2017-09-26T18:51:09  <sipa> Sentineo: we always make 8 outgoing connections - regardless of how many incoming connections you have
262 2017-09-26T18:51:14  *** AaronvanW has joined #bitcoin-core-dev
263 2017-09-26T18:51:29  <sipa> outgoing connections are far more sybil resistant, as their endpoints are under your own control
264 2017-09-26T18:52:00  <jtimon> I don't think that unusable, but ok
265 2017-09-26T18:52:02  <gmaxwell> Sentineo: "prefer inbound" ?!?! ... the inbound could all be a single badguy.  We consider inbounds irrelevant for security because of that.
266 2017-09-26T18:52:54  <Sentineo> yeah, did not thing of that. Was lookng at it the "i serve the network" point of view, which is stupid from security standpoing ...
267 2017-09-26T18:57:01  *** AaronvanW has quit IRC
268 2017-09-26T18:57:15  *** cdecker has quit IRC
269 2017-09-26T19:00:28  *** AaronvanW has joined #bitcoin-core-dev
270 2017-09-26T19:03:29  *** harrymm has quit IRC
271 2017-09-26T19:08:46  *** Chris_Stewart_5 has quit IRC
272 2017-09-26T19:10:05  <maaku> sipa: Bonjour/Zeroconf would make initial sync of new devices without blowing data caps much more idiot proof
273 2017-09-26T19:11:49  <maaku> it would also make sense to combine that with preferring making block requests to local peers during IBD
274 2017-09-26T19:12:48  *** RubenSomsen has quit IRC
275 2017-09-26T19:13:27  <sipa> the IBD sync will already prefer faster peers, and will disconnect peers that prevent it from going faster - so i hope no special logic for that is needed
276 2017-09-26T19:14:22  <sipa> i haven't looked at any of those local connection services
277 2017-09-26T19:16:38  <gmaxwell> they are awful zeroday magnets.
278 2017-09-26T19:16:44  <gmaxwell> of course.
279 2017-09-26T19:16:52  *** harrymm has joined #bitcoin-core-dev
280 2017-09-26T19:17:44  <maaku> sipa: the disconnect actually sounds somewhat dangerous. could I used fast connectivity and network attacks to partition someone off during IBD?
281 2017-09-26T19:18:51  <sipa> maaku: there is certainly an increased attack vector, but you'd at least need to combine it with other strategies to control the new connections being made (eclipse attacks etc)
282 2017-09-26T19:19:09  <sipa> maaku: in general i do think that partitioning during IBD is less of an issue - you know you're not caught up
283 2017-09-26T19:23:27  <TD-Linux> bonjour is a pretty sensible protocol for that
284 2017-09-26T19:23:40  <gmaxwell> you're still making progress and you already have your headers. There is an argument, however, that 'after' IBD we should dump most of our outbound peers.
285 2017-09-26T19:24:29  <gmaxwell> but that is complicated by there not being a clear 'after'.
286 2017-09-26T19:27:22  *** Chris_Stewart_5 has joined #bitcoin-core-dev
287 2017-09-26T19:36:45  <maaku> right
288 2017-09-26T19:38:38  *** Cheeseo has joined #bitcoin-core-dev
289 2017-09-26T19:40:17  *** timothy has quit IRC
290 2017-09-26T19:44:33  <gmaxwell> TD-Linux: things like bonjour don't help doing things like having connections inside amazon regions for hosts on aws, which have radically less expensive bandwidth than bandwidth to the internet.
291 2017-09-26T19:46:38  *** promag has joined #bitcoin-core-dev
292 2017-09-26T19:50:35  *** promag has quit IRC
293 2017-09-26T19:51:21  *** Chris_Stewart_5 has quit IRC
294 2017-09-26T19:53:57  *** cheese_ has joined #bitcoin-core-dev
295 2017-09-26T19:57:12  *** Cheeseo has quit IRC
296 2017-09-26T19:59:45  *** Miezel has quit IRC
297 2017-09-26T20:00:12  *** promag has joined #bitcoin-core-dev
298 2017-09-26T20:01:00  *** AaronvanW has quit IRC
299 2017-09-26T20:01:14  *** promag has quit IRC
300 2017-09-26T20:02:26  *** chjj has joined #bitcoin-core-dev
301 2017-09-26T20:04:29  <TD-Linux> gmaxwell, nope. I guess you could ship lists of these
302 2017-09-26T20:04:48  *** AaronvanW has joined #bitcoin-core-dev
303 2017-09-26T20:11:35  *** promag has joined #bitcoin-core-dev
304 2017-09-26T20:14:14  *** AaronvanW has quit IRC
305 2017-09-26T20:17:18  *** promag has quit IRC
306 2017-09-26T20:25:59  <bitcoin-git> [bitcoin] TheBlueMatt opened pull request #11406: Add state message print to AcceptBlock failure message. (master...2017-09-checkblock-fail-print) https://github.com/bitcoin/bitcoin/pull/11406
307 2017-09-26T20:29:22  *** meshcollider has joined #bitcoin-core-dev
308 2017-09-26T20:34:45  <jtimon> does anybody use -mempoolreplacement=0 ? would it make sense to do https://github.com/ElementsProject/elements/pull/248 in bitcoin ?
309 2017-09-26T20:36:15  *** promag has joined #bitcoin-core-dev
310 2017-09-26T20:44:40  *** Chris_Stewart_5 has joined #bitcoin-core-dev
311 2017-09-26T20:45:22  <luke-jr> jtimon: yes, people use it, and removing it would likely make a ruckus. There's also zero harm in leaving it.
312 2017-09-26T20:45:53  <jtimon> yeah, if people use it, no point in removing it
313 2017-09-26T20:47:45  *** Alina-malina has quit IRC
314 2017-09-26T20:48:56  *** promag has quit IRC
315 2017-09-26T20:49:51  *** Alina-malina has joined #bitcoin-core-dev
316 2017-09-26T20:54:20  *** promag has joined #bitcoin-core-dev
317 2017-09-26T20:56:18  <promag> achow101: in https://github.com/bitcoin/bitcoin/pull/10583/commits/8d5d4cf4c3077e8f3269fab3b98da7413e100081 commit message there is a double space. Also, Validateaddress should always be lowercase?
318 2017-09-26T20:59:50  <promag> jnewbery: so -deprecatedrpc can be used to enable some deprecated params? I thought is to enable the full (deprecated) rpc
319 2017-09-26T21:05:05  *** Cory has quit IRC
320 2017-09-26T21:07:59  <achow101> promag: who cares
321 2017-09-26T21:08:27  <achow101> promag: super nitty things like that don't really matter and are not particularly conducive to development
322 2017-09-26T21:10:47  *** Pasha has joined #bitcoin-core-dev
323 2017-09-26T21:11:28  <promag> that's ok, I was reading your code and figured you want to know about that if by any change you happen to fix something else
324 2017-09-26T21:11:37  <promag> *chance
325 2017-09-26T21:13:58  *** Pasha is now known as Cory
326 2017-09-26T21:14:11  <promag> talking about nitty things https://github.com/bitcoin/bitcoin/pull/11395#issuecomment-332308834
327 2017-09-26T21:15:27  <promag> luke-jr: if you run clang-format as per developer notes you will see those details
328 2017-09-26T21:16:12  *** BashCo has quit IRC
329 2017-09-26T21:17:12  *** Alina-malina has quit IRC
330 2017-09-26T21:18:48  <promag> if there is such guide, which took some community effort to write why not follow it? I would say top core developers should be the first to comply? It's not that it takes a lot of time doing so.
331 2017-09-26T21:20:20  *** Alina-malina has joined #bitcoin-core-dev
332 2017-09-26T21:20:44  *** BashCo has joined #bitcoin-core-dev
333 2017-09-26T21:24:57  *** RoyceX has joined #bitcoin-core-dev
334 2017-09-26T21:25:13  <bitcoin-git> [bitcoin] instagibbs opened pull request #11407: add functional test for mempoolreplacement command line arg (master...testmempoolreplacearg) https://github.com/bitcoin/bitcoin/pull/11407
335 2017-09-26T21:27:52  *** cheese_ has quit IRC
336 2017-09-26T21:34:31  *** Squidicuz has joined #bitcoin-core-dev
337 2017-09-26T21:34:57  *** Chris_Stewart_5 has quit IRC
338 2017-09-26T21:36:19  *** Guyver2 has quit IRC
339 2017-09-26T21:37:43  <sipa> promag: there is balance between costs and benefits of enforcing style
340 2017-09-26T21:38:06  <sipa> it adds overhead to both people who write patches and those who reviee
341 2017-09-26T21:41:48  *** Cory has quit IRC
342 2017-09-26T21:42:46  *** belcher has quit IRC
343 2017-09-26T21:44:52  *** belcher has joined #bitcoin-core-dev
344 2017-09-26T21:46:12  *** abpa has joined #bitcoin-core-dev
345 2017-09-26T21:48:44  *** Cory has joined #bitcoin-core-dev
346 2017-09-26T21:51:20  <jnewbery> promag: I personally think it's fine to use -deprecatedrpc to signal that you want to receive an RPC return object in the old format
347 2017-09-26T21:55:13  <promag> sipa: the problem is to find the sweet spot for everyone. Running clang-format-diff.py is not that big overhead is it? Anyway, I'll stop nitpicking.
348 2017-09-26T21:55:55  <promag> jnewbery: I guess so, and should tests all deprecated keys or one is enough?
349 2017-09-26T21:59:35  <bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc597bb895ef...2505c5c0a9f8
350 2017-09-26T21:59:35  <bitcoin-git> bitcoin/master 7b137ac Lucas Betschart: [Qt] Add delay before filtering transactions...
351 2017-09-26T21:59:36  <bitcoin-git> bitcoin/master 2505c5c Jonas Schnelli: Merge #11015: [Qt] Add delay before filtering transactions...
352 2017-09-26T22:00:06  <bitcoin-git> [bitcoin] jonasschnelli closed pull request #11015: [Qt] Add delay before filtering transactions (master...searchDelay) https://github.com/bitcoin/bitcoin/pull/11015
353 2017-09-26T22:01:50  <sipa> promag: i'm personally in favor of more strict style conventions, and would very much like the project to force one style... but that's just not feasible. nitpicking everything kills people's motivation to work
354 2017-09-26T22:06:11  <CryptAxe> Is it possible for nodes that don't understand the extended transaction serialization format for segwit to still validate blocks with those transactions inside of them?
355 2017-09-26T22:06:57  <goatpig> they cant check the scriptsigs
356 2017-09-26T22:07:14  <goatpig> but otherwise pre SW nodes can verify SW transactions
357 2017-09-26T22:08:12  <sipa> CryptAxe: sure, they just won't see the witnesses
358 2017-09-26T22:08:25  <sipa> (as peers will strip the witness before relaying to a non-segwit peer)
359 2017-09-26T22:08:57  <bitcoin-git> [bitcoin] PierreRochard opened pull request #11408: Trivial: Fix parameter name typo in ErasePurpose walletdb method (master...walletdb-typo) https://github.com/bitcoin/bitcoin/pull/11408
360 2017-09-26T22:09:31  *** promag has quit IRC
361 2017-09-26T22:09:31  <CryptAxe> So they can still deserialize the transaction? Cool, I think this part: https://github.com/bitcoin/bitcoin/blob/master/src/primitives/transaction.h#L204-L212 made me think that they would have an empty vin and it would be invalid
362 2017-09-26T22:09:40  *** maaku has left #bitcoin-core-dev
363 2017-09-26T22:10:01  <CryptAxe> Ahhh so the transactions are relayed in a different format to old nodes
364 2017-09-26T22:10:06  <CryptAxe> I didn't see that part
365 2017-09-26T22:10:10  <sipa> CryptAxe: if they see the witness, the serialization will very deliberately fail
366 2017-09-26T22:10:42  <sipa> CryptAxe: the reason segwit is a softfork is the fact that it is possible to strip out the witness without invalidating the transaction (from the perspective of old nodes)
367 2017-09-26T22:11:07  <CryptAxe> goatpig and sipa thank you, I have more reading to do now :)
368 2017-09-26T22:11:29  <sipa> it's probably better to think of the witness as not actually part of the transaction - it's just some auxilliary data - which _only for relay_ is relayed in a weird interspersed way inside the normal tx data
369 2017-09-26T22:12:06  <CryptAxe> sipa so the stripped format has some kind of dummy vin or something?
370 2017-09-26T22:12:10  <sipa> no
371 2017-09-26T22:12:14  *** promag has joined #bitcoin-core-dev
372 2017-09-26T22:12:21  *** cheese_ has joined #bitcoin-core-dev
373 2017-09-26T22:12:22  <sipa> the stripped format is just the old transaction format
374 2017-09-26T22:12:33  <sipa> it has inputs and outputs, and no witnesses
375 2017-09-26T22:12:43  <sipa> ah, i see what you mean
376 2017-09-26T22:12:49  <sipa> yes, the scriptSigs are empty
377 2017-09-26T22:12:58  <sipa> but the rest of the inputs is intact
378 2017-09-26T22:13:03  <CryptAxe> Thank you, a bit confused about all that
379 2017-09-26T22:13:35  *** promag has quit IRC
380 2017-09-26T22:15:32  *** RoyceX has quit IRC
381 2017-09-26T22:17:22  <CryptAxe> sipa without the scriptSigs why aren't the transactions invalid to the older nodes?
382 2017-09-26T22:17:42  <sipa> CryptAxe: because their scriptPubKeys don't require a scriptSig
383 2017-09-26T22:17:49  <CryptAxe> Ah!
384 2017-09-26T22:17:58  <sipa> (they have the form OP_0 <20-byte hash>)
385 2017-09-26T22:18:05  <CryptAxe> genius
386 2017-09-26T22:18:57  <sipa> if segwit is used in P2SH-wrapped form, the scriptPubKey is a P2SH output with Hash(OP_0 <20-byte hash>), and the scriptSig contains _only_ the redeemscript (which is equal to the actual OP_0 <20 byte hash>)
387 2017-09-26T22:19:07  *** Miezel has joined #bitcoin-core-dev
388 2017-09-26T22:21:24  <CryptAxe> Cool, thanks again guys!
389 2017-09-26T22:25:46  *** wittysense has quit IRC
390 2017-09-26T22:30:23  *** belcher has quit IRC
391 2017-09-26T22:36:05  *** Alina-malina has quit IRC
392 2017-09-26T22:38:25  *** Alina-malina has joined #bitcoin-core-dev
393 2017-09-26T22:53:41  *** Alina-malina has quit IRC
394 2017-09-26T22:57:30  *** Alina-malina has joined #bitcoin-core-dev
395 2017-09-26T23:04:35  *** abpa has quit IRC
396 2017-09-26T23:05:26  *** promag has joined #bitcoin-core-dev
397 2017-09-26T23:18:42  *** promag has quit IRC
398 2017-09-26T23:19:28  *** laurentmt has joined #bitcoin-core-dev
399 2017-09-26T23:20:17  *** belcher has joined #bitcoin-core-dev
400 2017-09-26T23:25:05  *** vicenteH has quit IRC
401 2017-09-26T23:29:01  *** Alina-malina has quit IRC
402 2017-09-26T23:30:58  *** Alina-malina has joined #bitcoin-core-dev
403 2017-09-26T23:39:40  *** circ-user-2HbxE has joined #bitcoin-core-dev
404 2017-09-26T23:43:22  *** circ-user-2HbxE has quit IRC
405 2017-09-26T23:57:20  *** laurentmt1 has joined #bitcoin-core-dev
406 2017-09-26T23:59:28  *** laurentmt has quit IRC
407 2017-09-26T23:59:28  *** laurentmt1 is now known as laurentmt