1 2018-11-20T00:05:07  *** rex4539 has quit IRC
  2 2018-11-20T00:09:36  *** dviola has joined #bitcoin-core-dev
  3 2018-11-20T00:14:18  *** kanzure_ is now known as kanzure
  4 2018-11-20T00:34:25  *** _cryptodesktop_i has joined #bitcoin-core-dev
  5 2018-11-20T00:42:09  *** queip has quit IRC
  6 2018-11-20T00:51:40  *** drexl_ has quit IRC
  7 2018-11-20T00:52:18  *** drexl has joined #bitcoin-core-dev
  8 2018-11-20T00:53:33  *** queip has joined #bitcoin-core-dev
  9 2018-11-20T01:08:54  *** _cryptodesktop_i has quit IRC
 10 2018-11-20T01:26:47  *** intcat has quit IRC
 11 2018-11-20T01:28:48  *** intcat has joined #bitcoin-core-dev
 12 2018-11-20T01:30:49  *** dviola has quit IRC
 13 2018-11-20T01:37:04  *** arubi_ has quit IRC
 14 2018-11-20T01:37:42  *** arubi has joined #bitcoin-core-dev
 15 2018-11-20T01:42:01  *** queip has quit IRC
 16 2018-11-20T01:44:28  *** Ga1aCt1Cz00_ has joined #bitcoin-core-dev
 17 2018-11-20T01:47:39  *** Ga1aCt1Cz00__ has quit IRC
 18 2018-11-20T01:49:09  *** queip has joined #bitcoin-core-dev
 19 2018-11-20T01:59:19  *** michaelsdunn1 has joined #bitcoin-core-dev
 20 2018-11-20T02:13:42  *** michaelsdunn1 has quit IRC
 21 2018-11-20T02:19:33  *** michaelsdunn1 has joined #bitcoin-core-dev
 22 2018-11-20T02:23:13  *** Chris_Stewart_5 has joined #bitcoin-core-dev
 23 2018-11-20T02:42:09  *** queip has quit IRC
 24 2018-11-20T02:45:58  *** michaelsdunn1 has quit IRC
 25 2018-11-20T02:55:49  *** queip has joined #bitcoin-core-dev
 26 2018-11-20T02:57:52  *** bitcoin-git has joined #bitcoin-core-dev
 27 2018-11-20T02:57:52  <bitcoin-git> [bitcoin] qqz898 opened pull request #14766: 0.17 (master...0.17) https://github.com/bitcoin/bitcoin/pull/14766
 28 2018-11-20T02:57:52  *** bitcoin-git has left #bitcoin-core-dev
 29 2018-11-20T02:59:02  *** bitcoin-git has joined #bitcoin-core-dev
 30 2018-11-20T02:59:03  <bitcoin-git> [bitcoin] qqz898 closed pull request #14766: 0.17 (master...0.17) https://github.com/bitcoin/bitcoin/pull/14766
 31 2018-11-20T02:59:03  *** bitcoin-git has left #bitcoin-core-dev
 32 2018-11-20T02:59:43  *** bitcoin-git has joined #bitcoin-core-dev
 33 2018-11-20T02:59:43  <bitcoin-git> [bitcoin] qqz898 closed pull request #14766: 0.17 (master...0.17) https://github.com/bitcoin/bitcoin/pull/14766
 34 2018-11-20T02:59:43  *** bitcoin-git has left #bitcoin-core-dev
 35 2018-11-20T03:07:26  *** AaronvanW has quit IRC
 36 2018-11-20T03:20:06  *** Chris_Stewart_5 has quit IRC
 37 2018-11-20T03:21:44  *** cjd has quit IRC
 38 2018-11-20T03:22:29  *** cjd has joined #bitcoin-core-dev
 39 2018-11-20T03:31:01  *** rh0nj has quit IRC
 40 2018-11-20T03:32:07  *** rh0nj has joined #bitcoin-core-dev
 41 2018-11-20T03:38:00  *** go1111111 has quit IRC
 42 2018-11-20T03:42:02  *** queip has quit IRC
 43 2018-11-20T03:51:16  *** go1111111 has joined #bitcoin-core-dev
 44 2018-11-20T03:52:53  *** queip has joined #bitcoin-core-dev
 45 2018-11-20T03:58:07  *** Chris_Stewart_5 has joined #bitcoin-core-dev
 46 2018-11-20T04:06:07  *** Chris_Stewart_5 has quit IRC
 47 2018-11-20T04:22:03  *** schnerch_ has joined #bitcoin-core-dev
 48 2018-11-20T04:24:51  *** schnerchi has quit IRC
 49 2018-11-20T04:28:50  *** cjd has quit IRC
 50 2018-11-20T04:28:58  *** cjd has joined #bitcoin-core-dev
 51 2018-11-20T04:32:38  *** gelmutshmidt has joined #bitcoin-core-dev
 52 2018-11-20T04:41:58  *** queip has quit IRC
 53 2018-11-20T04:49:57  *** queip has joined #bitcoin-core-dev
 54 2018-11-20T04:59:23  *** Eagle[TM] has joined #bitcoin-core-dev
 55 2018-11-20T05:01:53  *** EagleTM has quit IRC
 56 2018-11-20T05:12:12  <kallewoof> *headscratch* why would I get errors about "ifstream in fs not having a type" in fs.h (which I haven't touched) included from bitcoind.cpp (which I haven't touched)...?
 57 2018-11-20T05:12:30  <kallewoof> bottom of https://travis-ci.org/bitcoin/bitcoin/jobs/457291806
 58 2018-11-20T05:12:31  <sipa> kallewoof: master is broken
 59 2018-11-20T05:12:57  <kallewoof> sipa: oh
 60 2018-11-20T05:15:56  *** Ga1aCt1Cz00__ has joined #bitcoin-core-dev
 61 2018-11-20T05:19:23  *** Ga1aCt1Cz00_ has quit IRC
 62 2018-11-20T05:23:30  *** spinza has quit IRC
 63 2018-11-20T05:36:43  *** spinza has joined #bitcoin-core-dev
 64 2018-11-20T05:39:19  *** tryphe_ is now known as tryphe
 65 2018-11-20T05:42:53  *** bitcoin-git has joined #bitcoin-core-dev
 66 2018-11-20T05:42:53  <bitcoin-git> [bitcoin] kallewoof opened pull request #14768: revert removal of fstream.hpp header in fs.h (master...restore-fs-h-include-boost-fs-fstream) https://github.com/bitcoin/bitcoin/pull/14768
 67 2018-11-20T05:42:53  *** bitcoin-git has left #bitcoin-core-dev
 68 2018-11-20T05:50:30  *** bitcoin-git has joined #bitcoin-core-dev
 69 2018-11-20T05:50:30  <bitcoin-git> [bitcoin] MeshCollider closed pull request #14763: Remove filesystem/fstream from EXPECTED_BOOST_INCLUDES (master...2018/11/expected_boost_includes) https://github.com/bitcoin/bitcoin/pull/14763
 70 2018-11-20T05:50:30  *** bitcoin-git has left #bitcoin-core-dev
 71 2018-11-20T06:00:59  *** bitconner has quit IRC
 72 2018-11-20T06:17:21  *** bitconner has joined #bitcoin-core-dev
 73 2018-11-20T06:21:39  *** bitconner has quit IRC
 74 2018-11-20T06:42:03  *** queip has quit IRC
 75 2018-11-20T06:54:10  *** queip has joined #bitcoin-core-dev
 76 2018-11-20T07:20:08  *** Eagle[TM] has quit IRC
 77 2018-11-20T07:31:47  *** go1111111 has quit IRC
 78 2018-11-20T07:41:57  *** queip has quit IRC
 79 2018-11-20T07:47:25  *** go1111111 has joined #bitcoin-core-dev
 80 2018-11-20T07:52:36  *** face has joined #bitcoin-core-dev
 81 2018-11-20T07:55:48  *** queip has joined #bitcoin-core-dev
 82 2018-11-20T07:56:52  <gwillen> gmaxwell: can you explain to me what your comment on 14588 means? (I assume it's not for me, but for committers)
 83 2018-11-20T07:57:52  *** booyah has quit IRC
 84 2018-11-20T07:58:37  *** rex4539 has joined #bitcoin-core-dev
 85 2018-11-20T08:00:29  *** ghost43 has quit IRC
 86 2018-11-20T08:02:14  *** ghost43 has joined #bitcoin-core-dev
 87 2018-11-20T08:08:17  <sipa> gwillen: gmaxwell suggests we may want to include it in 0.17.1
 88 2018-11-20T08:08:47  <sipa> and no, not aimed at you (but feel to comment whether you think that's a good idea)
 89 2018-11-20T08:09:21  <gmaxwell> yeah, feel free to say if you think it should/shouldn't be.
 90 2018-11-20T08:19:11  *** ghost43 has quit IRC
 91 2018-11-20T08:20:40  *** booyah has joined #bitcoin-core-dev
 92 2018-11-20T08:24:40  *** ghost43 has joined #bitcoin-core-dev
 93 2018-11-20T08:28:49  *** EagleTM has joined #bitcoin-core-dev
 94 2018-11-20T08:41:34  *** EagleTM has quit IRC
 95 2018-11-20T08:42:01  *** queip has quit IRC
 96 2018-11-20T08:48:24  *** queip has joined #bitcoin-core-dev
 97 2018-11-20T08:55:54  <wumpus> meshcollider: I'm now
 98 2018-11-20T08:58:31  <wumpus> provoostenator: that really shouldn't have been merged before being sure about it, linters cause enough issues without feeding them false input
 99 2018-11-20T08:59:52  <wumpus> oh it doesn't change the linter config, just removes a header too much causing compile issues
100 2018-11-20T09:00:42  <wumpus> why is fstream only needed on some platforms?
101 2018-11-20T09:01:16  <wumpus> oh I see, it sometimes gets included through other headers
102 2018-11-20T09:07:12  *** bitcoin-git has joined #bitcoin-core-dev
103 2018-11-20T09:07:13  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/09f1d7fe7243...1b99d153d071
104 2018-11-20T09:07:13  <bitcoin-git> bitcoin/master e816b34 Karl-Johan Alm: revert removal of fstream.hpp header in fs.h...
105 2018-11-20T09:07:14  <bitcoin-git> bitcoin/master 1b99d15 Wladimir J. van der Laan: Merge #14768: revert removal of fstream.hpp header in fs.h...
106 2018-11-20T09:07:14  *** bitcoin-git has left #bitcoin-core-dev
107 2018-11-20T09:08:25  *** bitcoin-git has joined #bitcoin-core-dev
108 2018-11-20T09:08:25  <bitcoin-git> [bitcoin] laanwj closed pull request #14768: revert removal of fstream.hpp header in fs.h (master...restore-fs-h-include-boost-fs-fstream) https://github.com/bitcoin/bitcoin/pull/14768
109 2018-11-20T09:08:25  *** bitcoin-git has left #bitcoin-core-dev
110 2018-11-20T09:14:38  <wumpus> ouch, travis being down and it not being noticed is quite dangerous
111 2018-11-20T09:20:04  *** timothy has joined #bitcoin-core-dev
112 2018-11-20T09:20:31  <wumpus> luckily it was only a header issue then
113 2018-11-20T09:24:56  *** phwalkr has joined #bitcoin-core-dev
114 2018-11-20T09:28:15  *** belcher has quit IRC
115 2018-11-20T09:28:45  *** phwalkr has quit IRC
116 2018-11-20T09:33:56  *** phwalkr has joined #bitcoin-core-dev
117 2018-11-20T09:41:19  *** belcher has joined #bitcoin-core-dev
118 2018-11-20T09:42:05  *** queip has quit IRC
119 2018-11-20T09:43:19  *** intcat has quit IRC
120 2018-11-20T09:45:32  *** intcat has joined #bitcoin-core-dev
121 2018-11-20T09:48:34  *** queip has joined #bitcoin-core-dev
122 2018-11-20T10:03:36  *** setpill has joined #bitcoin-core-dev
123 2018-11-20T10:12:04  *** phwalkr has quit IRC
124 2018-11-20T10:16:47  <luke-jr> if Travis being down is dangerous, the real danger is the conclusion that things are being merged without real review :x
125 2018-11-20T10:18:49  <luke-jr> ie, it's being used as more than just a mere convenience
126 2018-11-20T10:22:51  *** michaelfolkson has joined #bitcoin-core-dev
127 2018-11-20T10:28:25  <wumpus> nah not fatally dangerous, the PR didn't go without review
128 2018-11-20T10:28:28  <promag> is there a way to make a long RPC call? like server side sleep?
129 2018-11-20T10:28:55  <wumpus> promag: yes, RPC calls can sleep as long as they want, although the client has to make sure they don't time out
130 2018-11-20T10:29:03  <promag> I thought we were doing blind merges :D
131 2018-11-20T10:29:29  <wumpus> it does hold up a RPC thread but there's other calls that can take a long time for example the utxo stats one
132 2018-11-20T10:29:52  <promag> wumpus: for testing I don't think we have?
133 2018-11-20T10:29:54  <wumpus> I think the mining RPC does long-polling
134 2018-11-20T10:30:00  <wumpus> and there's RPCs to wait for next block and such?
135 2018-11-20T10:30:12  <promag> ah!
136 2018-11-20T10:30:27  <promag> thank you sir
137 2018-11-20T10:30:33  <wumpus> np
138 2018-11-20T10:40:16  <promag> do we have tests to concurrently call RPC to the same node?
139 2018-11-20T10:41:58  *** queip has quit IRC
140 2018-11-20T10:53:38  *** queip has joined #bitcoin-core-dev
141 2018-11-20T10:54:33  *** spinza has quit IRC
142 2018-11-20T11:01:07  *** cryptoboy has joined #bitcoin-core-dev
143 2018-11-20T11:09:26  *** spinza has joined #bitcoin-core-dev
144 2018-11-20T11:15:50  *** Chris_Stewart_5 has joined #bitcoin-core-dev
145 2018-11-20T11:26:41  *** cryptoboy has quit IRC
146 2018-11-20T11:27:09  *** cryptoboy has joined #bitcoin-core-dev
147 2018-11-20T11:41:56  *** queip has quit IRC
148 2018-11-20T11:46:28  *** speedChicken_ has joined #bitcoin-core-dev
149 2018-11-20T11:48:01  *** rh0nj has quit IRC
150 2018-11-20T11:49:07  *** rh0nj has joined #bitcoin-core-dev
151 2018-11-20T11:49:48  *** AaronvanW has joined #bitcoin-core-dev
152 2018-11-20T11:53:13  *** michaelfolkson has quit IRC
153 2018-11-20T11:53:43  *** queip has joined #bitcoin-core-dev
154 2018-11-20T11:53:48  *** spinza has quit IRC
155 2018-11-20T11:57:00  *** spinza has joined #bitcoin-core-dev
156 2018-11-20T12:05:09  *** phwalkr has joined #bitcoin-core-dev
157 2018-11-20T12:14:16  <wumpus> promag: I don't think so
158 2018-11-20T12:26:29  *** phwalkr has quit IRC
159 2018-11-20T12:42:43  *** zhangzf has joined #bitcoin-core-dev
160 2018-11-20T12:51:32  *** rex4539 has quit IRC
161 2018-11-20T13:04:09  *** Chris_Stewart_5 has quit IRC
162 2018-11-20T13:10:26  *** rhavar has quit IRC
163 2018-11-20T13:14:03  <provoostenator> One reason we didn't notice Travis missing is that there's still a green checkbox from the other tools. That's strange, shouldn't Github show an error if any integration is missing / fails?
164 2018-11-20T13:17:49  *** intcat has quit IRC
165 2018-11-20T13:21:16  *** intcat has joined #bitcoin-core-dev
166 2018-11-20T13:33:26  <wumpus> normally yes
167 2018-11-20T13:39:05  <Varunram> only if it misses / fails though, not if it doesn't boot up the tests in the first place
168 2018-11-20T13:55:04  <wumpus> yes, that sound plausible
169 2018-11-20T14:16:31  *** bitcoin-git has joined #bitcoin-core-dev
170 2018-11-20T14:16:31  <bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/1b99d153d071...afa506f6ebeb
171 2018-11-20T14:16:32  <bitcoin-git> bitcoin/master c456fbd Russell Yanofsky: Refactor: Move m_db pointers into BerkeleyDatabase...
172 2018-11-20T14:16:32  <bitcoin-git> bitcoin/master 15c93f0 Chun Kuan Lee: wallet: Add trailing wallet.dat when detecting duplicate wallet if it's a directory.
173 2018-11-20T14:16:33  <bitcoin-git> bitcoin/master 5912031 Chun Kuan Lee: wallet: Create IsDatabaseLoaded function
174 2018-11-20T14:16:33  *** bitcoin-git has left #bitcoin-core-dev
175 2018-11-20T14:17:40  *** bitcoin-git has joined #bitcoin-core-dev
176 2018-11-20T14:17:40  <bitcoin-git> [bitcoin] laanwj closed pull request #14552: wallet: detecting duplicate wallet by comparing the db filename. (master...default-wallet-fix) https://github.com/bitcoin/bitcoin/pull/14552
177 2018-11-20T14:17:40  *** bitcoin-git has left #bitcoin-core-dev
178 2018-11-20T14:17:53  *** Chris_Stewart_5 has joined #bitcoin-core-dev
179 2018-11-20T14:29:12  *** booyah has quit IRC
180 2018-11-20T14:41:56  *** queip has quit IRC
181 2018-11-20T14:47:29  *** Guyver2 has joined #bitcoin-core-dev
182 2018-11-20T14:48:32  *** queip has joined #bitcoin-core-dev
183 2018-11-20T14:50:54  *** cryptoboy has quit IRC
184 2018-11-20T14:51:07  *** bitcoin-git has joined #bitcoin-core-dev
185 2018-11-20T14:51:08  <bitcoin-git> [bitcoin] scravy opened pull request #14770: Do not specify sudo in .travis (master...patch-1) https://github.com/bitcoin/bitcoin/pull/14770
186 2018-11-20T14:51:08  *** bitcoin-git has left #bitcoin-core-dev
187 2018-11-20T14:52:16  *** booyah has joined #bitcoin-core-dev
188 2018-11-20T14:53:22  *** cryptoboy has joined #bitcoin-core-dev
189 2018-11-20T15:07:10  *** setpill has quit IRC
190 2018-11-20T15:12:39  <promag> wumpus: I was waiting for https://github.com/bitcoin/bitcoin/pull/14552#pullrequestreview-176535062
191 2018-11-20T15:14:06  <wumpus> promag: huh didn't see that, you had already utACKed it
192 2018-11-20T15:14:57  <promag> yeah, that was a previous commit
193 2018-11-20T15:15:10  <wumpus> *sigh* ok
194 2018-11-20T15:15:16  <promag> how can I cancel an utACK?
195 2018-11-20T15:15:23  <wumpus> edit->delete it
196 2018-11-20T15:15:42  <promag> anyway, ken2812221_ ^
197 2018-11-20T15:15:47  *** LGzr has joined #bitcoin-core-dev
198 2018-11-20T15:16:02  <wumpus> I can revert it if that's really necessary
199 2018-11-20T15:16:09  <promag> I don't think so
200 2018-11-20T15:16:55  <promag> the mutex is for g_dbenvs, but there is no mutex for WalletEnvironment
201 2018-11-20T15:17:16  <promag> so I wonder if it should use the same
202 2018-11-20T15:23:01  <LGzr> Hola, all. I've a question about the p2sh example at https://bitcoin.org/en/developer-examples#p2sh-multisig; is there a more appropriate forum than here? (Question is: what purpose it serves that createmultisig uses public key for one, and addresses for the others)
203 2018-11-20T15:26:12  *** Victor_sueca has joined #bitcoin-core-dev
204 2018-11-20T15:27:21  *** Victorsueca has quit IRC
205 2018-11-20T15:33:32  *** michaelsdunn1 has joined #bitcoin-core-dev
206 2018-11-20T15:40:13  *** mr_paz has joined #bitcoin-core-dev
207 2018-11-20T15:42:05  *** queip has quit IRC
208 2018-11-20T15:47:42  <wumpus> LGzr: I think https://bitcoin.stackexchange.com/ is the best place to ask such things
209 2018-11-20T15:50:01  *** rh0nj has quit IRC
210 2018-11-20T15:50:24  <LGzr> Wumpus - thanks!
211 2018-11-20T15:51:07  *** rh0nj has joined #bitcoin-core-dev
212 2018-11-20T15:52:37  *** queip has joined #bitcoin-core-dev
213 2018-11-20T15:54:19  *** Victor_sueca has quit IRC
214 2018-11-20T15:56:35  *** Victorsueca has joined #bitcoin-core-dev
215 2018-11-20T16:09:42  *** dqx_ has joined #bitcoin-core-dev
216 2018-11-20T16:19:40  *** dqx has quit IRC
217 2018-11-20T16:20:23  <achow101> meshcollider: can you rebase #14491
218 2018-11-20T16:20:26  <gribble> https://github.com/bitcoin/bitcoin/issues/14491 | Allow descriptor imports with importmulti by MeshCollider · Pull Request #14491 · bitcoin/bitcoin · GitHub
219 2018-11-20T16:28:45  *** Victorsueca has quit IRC
220 2018-11-20T16:29:53  *** Victorsueca has joined #bitcoin-core-dev
221 2018-11-20T16:32:49  *** michaelfolkson has joined #bitcoin-core-dev
222 2018-11-20T16:40:50  *** Victorsueca has quit IRC
223 2018-11-20T16:41:57  *** queip has quit IRC
224 2018-11-20T16:43:00  *** promag has quit IRC
225 2018-11-20T16:46:42  *** rex4539 has joined #bitcoin-core-dev
226 2018-11-20T16:55:20  *** queip has joined #bitcoin-core-dev
227 2018-11-20T17:06:42  *** bitcoin-git has joined #bitcoin-core-dev
228 2018-11-20T17:06:42  <bitcoin-git> [bitcoin] MarcoFalke closed pull request #14755: Remove redundant readme info in /doc (master...issue-14639) https://github.com/bitcoin/bitcoin/pull/14755
229 2018-11-20T17:06:42  *** bitcoin-git has left #bitcoin-core-dev
230 2018-11-20T17:08:24  *** Victorsueca has joined #bitcoin-core-dev
231 2018-11-20T17:10:52  *** gelmutshmidt has quit IRC
232 2018-11-20T17:11:20  *** gelmutshmidt has joined #bitcoin-core-dev
233 2018-11-20T17:16:19  *** IRC-Source_71688 has joined #bitcoin-core-dev
234 2018-11-20T17:18:28  *** IRC-Source_71688 has quit IRC
235 2018-11-20T17:19:18  *** kexkey has joined #bitcoin-core-dev
236 2018-11-20T17:20:15  *** kelt has quit IRC
237 2018-11-20T17:21:14  *** michaelfolkson has quit IRC
238 2018-11-20T17:22:46  *** michaelfolkson has joined #bitcoin-core-dev
239 2018-11-20T17:23:06  *** jarthur has joined #bitcoin-core-dev
240 2018-11-20T17:31:02  *** cryptoboy has quit IRC
241 2018-11-20T17:31:09  *** kelt has joined #bitcoin-core-dev
242 2018-11-20T17:31:12  *** cryptoboy has joined #bitcoin-core-dev
243 2018-11-20T17:32:24  *** promag has joined #bitcoin-core-dev
244 2018-11-20T17:37:21  <provoostenator> I notice that dumpwallet (on a fresh wallet) contains two sections, I'm confused why:
245 2018-11-20T17:37:25  <provoostenator> cV6... 2018-11-20T17:23:30Z reserve=1 # addr=tb1q... hdkeypath=m/0'/0'/11'
246 2018-11-20T17:37:30  <provoostenator> 001433... 0 script=1 # addr=2Ms...
247 2018-11-20T17:39:52  <sipa> the script is the p2sh wrapper of that address, i suppose
248 2018-11-20T17:41:57  *** queip has quit IRC
249 2018-11-20T17:42:19  <provoostenator> Ah hence they start with 0014? So the first section is for legacy & native segwit, the second for p2sh wrapped segwit? But why don't the p2sh entries show an hdkeypath?
250 2018-11-20T17:44:25  <sipa> they're not keys
251 2018-11-20T17:45:00  <sipa> in the current wallet design, you don't import addresses
252 2018-11-20T17:45:15  <sipa> you import keys and scripts, and it figures out on its own what that implies for what is yours
253 2018-11-20T17:45:19  *** bitconner has joined #bitcoin-core-dev
254 2018-11-20T17:45:51  <sipa> in practice for the P2SH-P2WPKH address derived from a wallet key, IsMine works as follows:
255 2018-11-20T17:45:59  <sipa> * it sees a P2SH address, so it looks up the script for that scripthash
256 2018-11-20T17:46:36  <sipa> * it notices the script is a P2WPKH script, so it recurses by replacing it with the corresponding P2PKH script
257 2018-11-20T17:46:49  <sipa> * it notices it's a P2PKH script, so it looks up the pubkeyhash
258 2018-11-20T17:46:58  <sipa> * it finds the private key for it, so it's considered spendable
259 2018-11-20T17:47:15  <sipa> but the only thing in that whole sequence that has a keypath is the key at the end
260 2018-11-20T17:48:16  *** ddd has joined #bitcoin-core-dev
261 2018-11-20T17:49:38  *** zallarak has joined #bitcoin-core-dev
262 2018-11-20T17:49:42  *** bitconner has quit IRC
263 2018-11-20T17:50:03  *** rabidus has quit IRC
264 2018-11-20T17:50:09  *** ddd has quit IRC
265 2018-11-20T17:51:45  *** bitcoin-git has joined #bitcoin-core-dev
266 2018-11-20T17:51:46  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/afa506f6ebeb...6b90a2a0e065
267 2018-11-20T17:51:47  <bitcoin-git> bitcoin/master feeef7d Julian Fleischer: Do not specify sudo in .travis...
268 2018-11-20T17:51:47  <bitcoin-git> bitcoin/master 6b90a2a Wladimir J. van der Laan: Merge #14770: travis: Do not specify sudo in .travis...
269 2018-11-20T17:51:48  *** bitcoin-git has left #bitcoin-core-dev
270 2018-11-20T17:52:54  *** bitcoin-git has joined #bitcoin-core-dev
271 2018-11-20T17:52:54  <bitcoin-git> [bitcoin] laanwj closed pull request #14770: travis: Do not specify sudo in .travis (master...patch-1) https://github.com/bitcoin/bitcoin/pull/14770
272 2018-11-20T17:52:54  *** bitcoin-git has left #bitcoin-core-dev
273 2018-11-20T17:54:22  *** queip has joined #bitcoin-core-dev
274 2018-11-20T17:54:51  *** oneark has quit IRC
275 2018-11-20T17:57:12  *** rabidus has joined #bitcoin-core-dev
276 2018-11-20T17:58:07  *** Chris_Stewart_5 has quit IRC
277 2018-11-20T17:58:16  *** bitconner has joined #bitcoin-core-dev
278 2018-11-20T17:59:05  *** fabianfabian has joined #bitcoin-core-dev
279 2018-11-20T18:03:20  *** bitconner has quit IRC
280 2018-11-20T18:14:33  *** bitconner has joined #bitcoin-core-dev
281 2018-11-20T18:15:25  <provoostenator> sipa: thanks. Perhaps no point in cleaning up the current wallet, but dumpwallet could add comments to the script entries like "p2sh-segwit script for hdkeypath=m/0..."?
282 2018-11-20T18:16:41  <sipa> provoostenator: or we could dump the inferred descriptor for each entry :)
283 2018-11-20T18:16:42  *** promag has quit IRC
284 2018-11-20T18:19:00  <provoostenator> That makes sense on top of #11803, but more for the key entries than for the script entries.
285 2018-11-20T18:19:02  <gribble> https://github.com/bitcoin/bitcoin/issues/11803 | Bugfix: RPC/Wallet: Include HD key metadata in dumpwallet by luke-jr · Pull Request #11803 · bitcoin/bitcoin · GitHub
286 2018-11-20T18:19:16  <sipa> provoostenator: no, that makes no sense
287 2018-11-20T18:19:23  <sipa> keys are keys, not addresses
288 2018-11-20T18:19:35  <provoostenator> But they're supposed to be interpreted as combo() descriptors
289 2018-11-20T18:19:46  <sipa> they participate in all scripts that use them
290 2018-11-20T18:19:47  <provoostenator> Depending on wallet age
291 2018-11-20T18:19:55  <sipa> and we can't even efficiently iterate those
292 2018-11-20T18:20:21  <sipa> yes, they also (for now) imply a combo descriptor on their own
293 2018-11-20T18:20:29  <sipa> but that's not very informative
294 2018-11-20T18:20:56  <sipa> "this key acts in all the way it itself can be an address!"
295 2018-11-20T18:21:02  <provoostenator> It's perhaps more informative than the current address= comment which is misleading.
296 2018-11-20T18:21:15  <sipa> ah, i see
297 2018-11-20T18:21:29  <sipa> it could list all addresses, or a combo descriptor - that's fair
298 2018-11-20T18:21:52  <sipa> but for scripts it's actually interesting, as it will tell which which keys it is related to
299 2018-11-20T18:23:14  <provoostenator> Yes, but the script is only relevant for p2sh wrapped segwit, right? (in a default new wallet)
300 2018-11-20T18:23:50  <sipa> sure
301 2018-11-20T18:23:55  <sipa> or anything you manually imported
302 2018-11-20T18:24:02  <sipa> (multisig etc)
303 2018-11-20T18:24:49  <provoostenator> In that case I think it's more clear for the comment to indicate that it's only there for p2sh wrapped segwit support, i.e. those entries are redundant once descriptors take over.
304 2018-11-20T18:25:15  <sipa> dumpwallet as a whole will need to be revamped post-descriptors
305 2018-11-20T18:25:39  <sipa> i don't understand what you're trying to argue for
306 2018-11-20T18:25:56  <sipa> scripts are not just there for p2sh wrapped segwit
307 2018-11-20T18:25:58  <sipa> they are there for p2sh and p2wsh everything
308 2018-11-20T18:26:00  <provoostenator> Indeed, but it might be useful to get it good enough so it's easier to test things (like upgrade paths) once we have descriptor functionality complete.
309 2018-11-20T18:26:20  <provoostenator> Right now it's really to inspect what is actually in a wallet.
310 2018-11-20T18:26:24  <provoostenator> *really hard
311 2018-11-20T18:28:04  <provoostenator> The comment would only be there for scripts that relate to the standard wallet, not any imported / generated p2(w)sh stuff.
312 2018-11-20T18:28:23  <sipa> we don't know that
313 2018-11-20T18:28:27  <sipa> there is just a bunch of scripts
314 2018-11-20T18:28:47  <sipa> they're not in any way linked to other wallet information
315 2018-11-20T18:29:33  *** Chris_Stewart_5 has joined #bitcoin-core-dev
316 2018-11-20T18:30:22  <provoostenator> You could re-match them using the private keys from all keys with an hdmasterkeyid?
317 2018-11-20T18:31:47  <sipa> or you could just print their inferred descriptor
318 2018-11-20T18:31:52  <sipa> which works for everything
319 2018-11-20T18:31:57  <sipa> and is efficient
320 2018-11-20T18:32:25  <sipa> even that may be confusing, though
321 2018-11-20T18:32:50  <sipa> as a p2sh-p2wsh script will have two entries
322 2018-11-20T18:32:50  <sipa> and we don't dump watched scripts
323 2018-11-20T18:33:07  * sipa would like to see dumpwallet die
324 2018-11-20T18:34:06  <provoostenator> I think dumpwallet can be useful for testing transition to descriptor based wallets, but I'm fine with making it die after that :-)
325 2018-11-20T18:34:54  <sipa> yeah, of course
326 2018-11-20T18:37:48  <provoostenator> sipa: about REST https://github.com/bitcoin/bitcoin/pull/12040#discussion_r235092254
327 2018-11-20T18:38:02  <provoostenator> Look for my example "do some wallet operation on wallet"
328 2018-11-20T18:38:40  <provoostenator> It passes an Authorization: Basic param with the request, which contains the RPC username and password in  some base64 converted form. Afaik it has full RPC access.
329 2018-11-20T18:38:47  <provoostenator> That's not because of this PR.
330 2018-11-20T18:39:20  <provoostenator> (or if it is, that's very underhanded code)
331 2018-11-20T18:40:20  <sipa> provoostenator: yes, i see that, but that is just  the RPC interface
332 2018-11-20T18:40:34  <provoostenator> Oh wait, but then what is the -rest parameter doing?
333 2018-11-20T18:40:37  <sipa> not the REST interface, which uses .../rest/... URLs
334 2018-11-20T18:40:48  <provoostenator> -rpcport=8080 is just cancelling my -rest=1?
335 2018-11-20T18:41:01  <sipa> no
336 2018-11-20T18:41:07  <sipa> -rest enables the REST interface
337 2018-11-20T18:41:29  <sipa> that doesn't mean you can't use RPC anymore
338 2018-11-20T18:42:00  *** queip has quit IRC
339 2018-11-20T18:42:10  <sipa> ah, -rpcport is somewhat confusingly named i guess, as that option indeed does affect both rpc and rest
340 2018-11-20T18:42:24  *** LGzr has quit IRC
341 2018-11-20T18:42:46  <provoostenator> Ah, it seems very unhealthy that RPC and REST are so intertwined.
342 2018-11-20T18:42:59  <provoostenator> Shouldn't REST just launch it's own server?
343 2018-11-20T18:43:08  <sipa> why?
344 2018-11-20T18:43:24  <sipa> they're entirely separate, apart from both being HTTP
345 2018-11-20T18:43:54  <sipa> rpcport should be named httpport i guess, but it's a bit late for that
346 2018-11-20T18:43:58  <provoostenator> I might not want to expose my RPC port to the internet
347 2018-11-20T18:44:17  <provoostenator> Oh wait, it doesn't move the RPC port? Ok, that is _really_ confusing :-)
348 2018-11-20T18:44:25  <sipa> sigh
349 2018-11-20T18:44:44  <sipa> rpcport sets the HTTP listen port
350 2018-11-20T18:44:45  <sipa> both RPC and REST use HTTP
351 2018-11-20T18:44:53  <sipa> and you shouldn't expose either to the internet
352 2018-11-20T18:45:06  <sipa> only the P2P interface is designed to be DoS resilient
353 2018-11-20T18:45:53  <provoostenator> Ok, but DoS is a different threat than someone gaining RPC control.
354 2018-11-20T18:46:02  <sipa> sure
355 2018-11-20T18:46:12  <sipa> but neither are designed to be exposed to the internet
356 2018-11-20T18:47:30  *** queip has joined #bitcoin-core-dev
357 2018-11-20T18:48:43  *** dviola has joined #bitcoin-core-dev
358 2018-11-20T18:53:09  *** bitconner has quit IRC
359 2018-11-20T18:54:34  <wumpus> it would be fully possible to start multiple http servers, and have rpc run on one and rest on the other, though I doubt it's going to be a feature that sees much use
360 2018-11-20T18:55:53  <provoostenator> I think it would be less confusing if it was a seperate server with its own port, but for this ticket that doesn't matter. The trick is just to only add CORS stuff to /rest/* so we don't have to worry about the phishing scenario I described.
361 2018-11-20T18:55:55  <wumpus> *if* you want to expose anything to the internet you're going to want to put an nginx server or such in front, which would filter what URLs are allowed
362 2018-11-20T18:56:55  <wumpus> I don't see how it's confusing, it has been like this since the beginning and AFAIK it's all documented *shrugs*
363 2018-11-20T18:57:08  <sipa> provoostenator: why do you need CORS on REST?
364 2018-11-20T18:57:24  <wumpus> that sounds really scary
365 2018-11-20T18:58:01  <provoostenator> Well that's the point of that PR, which we could debate in general.
366 2018-11-20T18:58:30  <sipa> i thought it was about something something browser security (i don't understand browsers, this may be my misunderstanding)
367 2018-11-20T18:58:45  <sipa> but for REST there is no security needed
368 2018-11-20T18:59:09  <provoostenator> At the moment browsers refuse to connect to the REST RPC via JSON.
369 2018-11-20T18:59:10  <wumpus> it'd be about letting websites make requests to the local bitcoind instance, hence 'cross-origin'
370 2018-11-20T18:59:21  <wumpus> I think that's completely ill-advices
371 2018-11-20T18:59:46  <sipa> ah, it's permitting something, not denying something
372 2018-11-20T18:59:49  <wumpus> right
373 2018-11-20T18:59:51  <provoostenator> Yes
374 2018-11-20T18:59:55  <sipa> i see
375 2018-11-20T19:00:06  <provoostenator> I don't see any downside with read-only access.
376 2018-11-20T19:00:14  <sipa> permitting sites to access REST sounds much less scary than letting them access RPC though
377 2018-11-20T19:00:22  <wumpus> it increases the attack surface to any random website
378 2018-11-20T19:00:38  <provoostenator> That's true.
379 2018-11-20T19:00:48  <wumpus> what if it's not really read-only but there is some bug in the REST code or a heartbleed-kind of bug that exposes memory and private keys or whoknowswhat
380 2018-11-20T19:01:00  <wumpus> I really prefer not having that kind of stuff in bitcoind, sorry
381 2018-11-20T19:01:23  <provoostenator> I guess that begs the question why this REST service exists in the first place.
382 2018-11-20T19:01:36  <wumpus> it's a fast way of accessing bitcoind's state
383 2018-11-20T19:02:01  <wumpus> no need for JSON parsing and formatting, it can give various binary objects directly
384 2018-11-20T19:02:22  <wumpus> also it can work without autentication for *local* programs, which is useful
385 2018-11-20T19:02:52  <jarthur> Yea, and that's a big plus. In sidecar services that need to parse the blockchain from a node, so much CPU time is spent on JSON parsing when using the RPC api.
386 2018-11-20T19:03:04  <wumpus> that doesn't mean you'd want to either expose it to the whole internet, or to every website in a browser
387 2018-11-20T19:04:43  <wumpus> that'd instantly lift any vulnerability from local-only and limited scope to, pretty much, rce
388 2018-11-20T19:04:51  <wumpus> same for DoS
389 2018-11-20T19:04:59  <provoostenator> So it's for fast queries by semi-trusted software.
390 2018-11-20T19:05:04  <wumpus> yes
391 2018-11-20T19:05:52  <provoostenator> That makes sense. It might be good to document that intention to set expectations: https://github.com/bitcoin/bitcoin/blob/master/doc/REST-interface.md
392 2018-11-20T19:05:56  <wumpus> or say, at most internal to your company if you bind on a local network interface and know what you're doing
393 2018-11-20T19:07:17  <wumpus> bitcoind's P2P interface is the only interface that is hardened for public internet access, and even there improvements are certainly possible
394 2018-11-20T19:07:17  <sipa> provoostenator: that sounds like a good idea
395 2018-11-20T19:07:38  <wumpus> yes
396 2018-11-20T19:07:58  <wumpus> would be good to document that better
397 2018-11-20T19:08:55  <sipa> wumpus: so, in fairness, the issue of exposing REST to browsers (or any local program running untrusted code) already exists, and it's only through browsers' self-restraint that this is somewhat curbed
398 2018-11-20T19:09:52  <sipa> i don't think CORS makes things any worse, it's just a question of whether we should treat browsers accessing REST as a supported/recommended practice
399 2018-11-20T19:10:10  <sipa> and in general i think the answer is no, but perhaps for a company's internal site it makes sense
400 2018-11-20T19:11:10  <gmaxwell> at least so long as REST is merely a DOS/RCE vector and not intentionally given access to anything too interesting it's not as big a deal.
401 2018-11-20T19:11:22  <gmaxwell> don't want to repeat ethereum's mistakes.
402 2018-11-20T19:13:10  <wumpus> sipa: *everything* is due to browser's self restraint, any security at all, if javascript would run unrestricted it would be really bad
403 2018-11-20T19:14:20  <wumpus> and sure, there could be an option to add CORS headers for specific sites
404 2018-11-20T19:14:35  <sipa> the PR lets you configure domains
405 2018-11-20T19:14:37  <wumpus> if anyone is going to use that and test that
406 2018-11-20T19:14:48  <wumpus> I'm just afraid of scope creep, as you know
407 2018-11-20T19:15:24  <sipa> wumpus: i understand, but my point is that this isn't really restricted to browsers; the issue exists equally for say someone running an untrusted VM image that makes a connection to the host network
408 2018-11-20T19:15:28  <wumpus> look, there's this concern we haven't even ever *thought* about and suddenly it needs to be considered in bitcoin core
409 2018-11-20T19:15:58  <sipa> so our job should be to make sure that by default, these things aren't a concern, and document them well
410 2018-11-20T19:16:30  <wumpus> yes, okay, the general case where peopel run untrusted software that can access local network, it's not possible to protect against that
411 2018-11-20T19:16:39  <provoostenator> Someone who really wants this could add an nginx server that just adds the header etc.
412 2018-11-20T19:16:43  <sipa> (i'm not arguing in favor of the CORS PR; I don't understand the use cases well enough, it just sounds to me that if remote access from local software is a problem, the solution is elsewhere)
413 2018-11-20T19:17:10  <wumpus> remote access from random websites that the user opens in their browser is a problem
414 2018-11-20T19:17:32  <wumpus> 'remote' I mean, it's effectively local, the browser protects that using CORS
415 2018-11-20T19:18:07  <sipa> i mean things like by default not exposing REST, not exposing RPC port publicly, not having a trivial RPC username/password are all protections against this
416 2018-11-20T19:18:10  <provoostenator> Scope creep is a nice thing to prevent. Even having to explain in documentation what the pitfalls of CORS are, and keep track of that, is overhead.
417 2018-11-20T19:18:19  <gmaxwell> sipa: to be fair 'VM sandbox that has network access as localhost' is kind of a special case that applies to FAR fewer users than "I run a browser"
418 2018-11-20T19:19:04  <sipa> wumpus: also, unix domain sockets instead of TCP/IP for RPC would be an improvement
419 2018-11-20T19:19:27  <sipa> so it's at least restricted to the same user on the same machine
420 2018-11-20T19:19:48  <jarthur> Yea, someone just needs to find time to finish that out.
421 2018-11-20T19:20:46  <jarthur> Maybe I'll get some time this weekend if no one else is working on it
422 2018-11-20T19:21:02  <wumpus> yes unix domain sockets would be great, I should rebase that some time
423 2018-11-20T19:21:08  *** timothy has quit IRC
424 2018-11-20T19:21:37  <wumpus> or someone else :)
425 2018-11-20T19:21:57  *** dqx has joined #bitcoin-core-dev
426 2018-11-20T19:25:33  *** dqx_ has quit IRC
427 2018-11-20T19:30:01  *** rh0nj has quit IRC
428 2018-11-20T19:31:08  *** rh0nj has joined #bitcoin-core-dev
429 2018-11-20T19:35:49  <provoostenator> Unix domain sockets also to get data in other formats than JSON (so you don't need the REST API at all)?
430 2018-11-20T19:37:20  *** Tennis has joined #bitcoin-core-dev
431 2018-11-20T19:39:39  *** dqx_ has joined #bitcoin-core-dev
432 2018-11-20T19:40:50  <wumpus> it'd be the same http server just over a different socket
433 2018-11-20T19:41:01  <wumpus> not a completely different API
434 2018-11-20T19:41:24  *** dqx_ has quit IRC
435 2018-11-20T19:41:56  *** dqx_ has joined #bitcoin-core-dev
436 2018-11-20T19:41:56  *** queip has quit IRC
437 2018-11-20T19:42:09  *** dqx has quit IRC
438 2018-11-20T19:43:43  *** bitconner has joined #bitcoin-core-dev
439 2018-11-20T19:44:43  *** michaelsdunn1 has quit IRC
440 2018-11-20T19:44:57  *** Chris_Stewart_5 has quit IRC
441 2018-11-20T19:46:12  *** michaelsdunn1 has joined #bitcoin-core-dev
442 2018-11-20T19:46:12  *** michaelsdunn1 has quit IRC
443 2018-11-20T19:46:12  *** michaelsdunn1 has joined #bitcoin-core-dev
444 2018-11-20T19:48:19  *** Chris_Stewart_5 has joined #bitcoin-core-dev
445 2018-11-20T19:48:39  *** bitconner has quit IRC
446 2018-11-20T19:48:53  *** zallarak has quit IRC
447 2018-11-20T19:49:33  *** queip has joined #bitcoin-core-dev
448 2018-11-20T19:50:22  *** dqx_ has quit IRC
449 2018-11-20T19:54:49  *** mr_paz has quit IRC
450 2018-11-20T19:58:36  *** kelt has quit IRC
451 2018-11-20T20:06:05  *** bitconner has joined #bitcoin-core-dev
452 2018-11-20T20:06:08  *** bitcoin-git has joined #bitcoin-core-dev
453 2018-11-20T20:06:09  <bitcoin-git> [bitcoin] vim88 closed pull request #14753: Refactor: Changes postincrement to preincrement for iterator in for loops in src/wallet files (master...postincrement_to_preincrement_src_wallet) https://github.com/bitcoin/bitcoin/pull/14753
454 2018-11-20T20:06:09  *** bitcoin-git has left #bitcoin-core-dev
455 2018-11-20T20:06:44  *** promag has joined #bitcoin-core-dev
456 2018-11-20T20:07:21  *** asoltys has quit IRC
457 2018-11-20T20:16:17  <jnewbery> promag: > do we have tests to concurrently call RPC to the same node?
458 2018-11-20T20:16:40  <jnewbery> yes: check mining_getblocktemplate_longpoll.py
459 2018-11-20T20:16:51  <promag> jnewbery: hi, yap saw that
460 2018-11-20T20:17:11  <promag> see #14670
461 2018-11-20T20:17:18  <gribble> https://github.com/bitcoin/bitcoin/issues/14670 | http: Fix HTTP server shutdown by promag · Pull Request #14670 · bitcoin/bitcoin · GitHub
462 2018-11-20T20:17:18  <jnewbery> If you want to do similar in a new test, I recommend you take the LongPoll thread class and lift it into TestNode
463 2018-11-20T20:17:47  <promag> https://github.com/bitcoin/bitcoin/pull/14670/files#diff-9ee18e00fd5f5f1444ba12e8e1378e6a
464 2018-11-20T20:18:07  <promag> leave a comment there please, bbl
465 2018-11-20T20:33:27  *** arubi has quit IRC
466 2018-11-20T20:34:24  *** arubi has joined #bitcoin-core-dev
467 2018-11-20T20:34:31  *** andrew79 has joined #bitcoin-core-dev
468 2018-11-20T20:34:57  *** ezzzy has joined #bitcoin-core-dev
469 2018-11-20T20:41:58  *** queip has quit IRC
470 2018-11-20T20:46:45  *** queip has joined #bitcoin-core-dev
471 2018-11-20T20:47:44  *** ezzzy has quit IRC
472 2018-11-20T20:49:43  *** michaelsdunn1 has quit IRC
473 2018-11-20T20:52:05  *** ezzzy has joined #bitcoin-core-dev
474 2018-11-20T20:58:44  *** wpaulino has quit IRC
475 2018-11-20T20:58:44  *** wpaulino has joined #bitcoin-core-dev
476 2018-11-20T20:59:12  *** michaelsdunn1 has joined #bitcoin-core-dev
477 2018-11-20T20:59:12  *** michaelsdunn1 has quit IRC
478 2018-11-20T20:59:12  *** michaelsdunn1 has joined #bitcoin-core-dev
479 2018-11-20T21:01:24  *** ezzzy has quit IRC
480 2018-11-20T21:02:21  *** michaelsdunn1 has quit IRC
481 2018-11-20T21:03:51  *** michaelsdunn1 has joined #bitcoin-core-dev
482 2018-11-20T21:04:43  *** dviola has quit IRC
483 2018-11-20T21:08:27  *** michaelsdunn1 has quit IRC
484 2018-11-20T21:16:17  *** michaelsdunn1 has joined #bitcoin-core-dev
485 2018-11-20T21:19:48  *** gelmutshmidt has quit IRC
486 2018-11-20T21:24:43  *** Jbaczuk_ has quit IRC
487 2018-11-20T21:34:15  *** owowo has quit IRC
488 2018-11-20T21:34:22  *** andrew79 has quit IRC
489 2018-11-20T21:39:07  *** owowo has joined #bitcoin-core-dev
490 2018-11-20T21:40:38  *** cryptoboy has quit IRC
491 2018-11-20T21:40:43  *** jrayhawk_ is now known as jrayhawk
492 2018-11-20T21:41:56  *** queip has quit IRC
493 2018-11-20T21:45:47  *** elichai2 has quit IRC
494 2018-11-20T21:52:32  *** shesek has joined #bitcoin-core-dev
495 2018-11-20T21:52:32  *** shesek has joined #bitcoin-core-dev
496 2018-11-20T21:54:13  *** bitconner has quit IRC
497 2018-11-20T21:54:37  *** tmz24 has joined #bitcoin-core-dev
498 2018-11-20T21:55:03  *** tmz24 has quit IRC
499 2018-11-20T21:57:49  *** queip has joined #bitcoin-core-dev
500 2018-11-20T22:00:47  *** justanotheruser has quit IRC
501 2018-11-20T22:15:51  *** hex17or has joined #bitcoin-core-dev
502 2018-11-20T22:18:52  *** hex17or has quit IRC
503 2018-11-20T22:20:08  *** promag has quit IRC
504 2018-11-20T22:21:38  *** hex17or has joined #bitcoin-core-dev
505 2018-11-20T22:21:45  *** rex4539 has quit IRC
506 2018-11-20T22:33:05  *** spinza has quit IRC
507 2018-11-20T22:36:19  *** spinza has joined #bitcoin-core-dev
508 2018-11-20T22:37:44  *** Chris_Stewart_5 has quit IRC
509 2018-11-20T22:40:05  *** hex17or has quit IRC
510 2018-11-20T22:41:56  *** queip has quit IRC
511 2018-11-20T22:42:22  *** hex17or has joined #bitcoin-core-dev
512 2018-11-20T22:46:55  *** hex17or has quit IRC
513 2018-11-20T22:50:58  *** hex17or has joined #bitcoin-core-dev
514 2018-11-20T22:53:04  *** hex17or has quit IRC
515 2018-11-20T22:54:52  *** queip has joined #bitcoin-core-dev
516 2018-11-20T22:58:56  *** justanotheruser has joined #bitcoin-core-dev
517 2018-11-20T23:04:16  *** iphelix has quit IRC
518 2018-11-20T23:04:24  *** iphelix- has joined #bitcoin-core-dev
519 2018-11-20T23:04:48  *** iphelix- is now known as iphelix
520 2018-11-20T23:05:52  *** iphelix is now known as Guest19336
521 2018-11-20T23:07:42  *** jamesob has quit IRC
522 2018-11-20T23:07:54  *** jamesob has joined #bitcoin-core-dev
523 2018-11-20T23:11:34  *** bitconner has joined #bitcoin-core-dev
524 2018-11-20T23:28:07  *** dqx has joined #bitcoin-core-dev
525 2018-11-20T23:40:58  *** fabianfabian has quit IRC
526 2018-11-20T23:41:59  *** bitcoin-git has joined #bitcoin-core-dev
527 2018-11-20T23:41:59  <bitcoin-git> [bitcoin] MarcoFalke opened pull request #14771: test: Add BOOST_REQUIRE to getters returning optional (master...Mf1811-testNoDiscard) https://github.com/bitcoin/bitcoin/pull/14771
528 2018-11-20T23:41:59  *** bitcoin-git has left #bitcoin-core-dev
529 2018-11-20T23:42:05  *** queip has quit IRC
530 2018-11-20T23:53:39  *** michaelfolkson has joined #bitcoin-core-dev
531 2018-11-20T23:54:00  *** queip has joined #bitcoin-core-dev