ChanServ changed the topic of #bitcoin-core-dev to: Bitcoin Core development discussion and commit log | Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Weekly Meeting Thursday @ 14:00 UTC | Meeting topics http://gnusha.org/bitcoin-core-dev/proposedmeetingtopics.txt
zeropoint has quit [Quit: leaving]
marsupialSoup has joined #bitcoin-core-dev
marsupialSoup has quit [Ping timeout: 268 seconds]
robobub has joined #bitcoin-core-dev
___nick___ has quit [Quit: https://quassel-irc.org - Chat comfortably. Anywhere.]
___nick___ has joined #bitcoin-core-dev
___nick___ has quit [Client Quit]
___nick___ has joined #bitcoin-core-dev
vyHamii has joined #bitcoin-core-dev
Torr has quit [Quit: Torr]
andrewtoth_ has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
test__ has joined #bitcoin-core-dev
test_ has quit [Ping timeout: 240 seconds]
andrewtoth_ has quit [Ping timeout: 260 seconds]
cmirror has quit [Remote host closed the connection]
cmirror has joined #bitcoin-core-dev
pyth_ has quit [Ping timeout: 268 seconds]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 240 seconds]
SpellChecker has quit [Quit: bye]
SpellChecker has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
dongcarl has quit [Ping timeout: 256 seconds]
SpellChecker has joined #bitcoin-core-dev
dongcarl has joined #bitcoin-core-dev
bitdex has joined #bitcoin-core-dev
salvatoshi has joined #bitcoin-core-dev
Lockesmith has joined #bitcoin-core-dev
Guyver2 has joined #bitcoin-core-dev
Guest74 has joined #bitcoin-core-dev
Guest74 has quit [Client Quit]
abubakarsadiq has joined #bitcoin-core-dev
___nick___ has quit [Ping timeout: 264 seconds]
BrandonOdiwuor has joined #bitcoin-core-dev
BrandonOdiwuor has quit [Client Quit]
BrandonOdiwuor has joined #bitcoin-core-dev
midnight_ has quit [Ping timeout: 264 seconds]
cold has quit [Ping timeout: 260 seconds]
midnight has joined #bitcoin-core-dev
cold has joined #bitcoin-core-dev
Guyver2 has left #bitcoin-core-dev [Closing Window]
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/ff7d2054c4f1...f47cd649e970
<bitcoin-git> bitcoin/master c3a5e8a Cory Fields: build: re-enable deprecated warning copy
<bitcoin-git> bitcoin/master f47cd64 merge-script: Merge bitcoin/bitcoin#30236: build: re-enable deprecated warning copy
<bitcoin-git> [bitcoin] fanquake merged pull request #30236: build: re-enable deprecated warning copy (master...reenable-deprecated-copy) https://github.com/bitcoin/bitcoin/pull/30236
<bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/f47cd649e970...1040a1fc807e
<bitcoin-git> bitcoin/master 232928b fanquake: build: no-longer allow GCC-10 in C++20 check
<bitcoin-git> bitcoin/master 1040a1f merge-script: Merge bitcoin/bitcoin#30228: build: no-longer allow GCC-10 in C++20 check
<bitcoin-git> [bitcoin] fanquake merged pull request #30228: build: no-longer allow GCC-10 in C++20 check (master...no_longer_allow_gcc_10) https://github.com/bitcoin/bitcoin/pull/30228
TallTim_ is now known as TallTim
pyth has joined #bitcoin-core-dev
Guest42 has joined #bitcoin-core-dev
Guest42 has quit [Client Quit]
<bitcoin-git> [bitcoin] AngusP opened pull request #30237: test: Add Compact Block Encoding test `ReceiveWithExtraTransactions` covering non-empty `extra_txn` (master...2024-06-cmpctblk-extra-txn-test) https://github.com/bitcoin/bitcoin/pull/30237
SpellChecker_ has joined #bitcoin-core-dev
SpellChecker has quit [Remote host closed the connection]
cygnet3 has joined #bitcoin-core-dev
andrewtoth_ has joined #bitcoin-core-dev
bitdex has quit [Quit: = ""]
<bitcoin-git> [bitcoin] glozow closed pull request #27591: rpc: distinguish between vsize and sigop-adjusted mempool vsize (master...2023-05-mempool-vsize) https://github.com/bitcoin/bitcoin/pull/27591
andrewtoth_ has quit [Ping timeout: 260 seconds]
pyth has quit [Read error: Connection reset by peer]
pyth has joined #bitcoin-core-dev
andrewtoth_ has joined #bitcoin-core-dev
Emc99 has joined #bitcoin-core-dev
<achow101> #startmeeting
<sdaftuar> hi
<glozow> hi
<achow101> #bitcoin-core-dev Meeting: achow101 _aj_ amiti ariard aureleoules b10c BlueMatt brunoerg cfields darosior dergoegge dongcarl fanquake fjahr furszy gleb glozow hebasto instagibbs jamesob jarolrod jonatack josibake kallewoof kanzure kouloumos kvaciral laanwj LarryRuane lightlike luke-jr MacroFake Murch phantomcircuit pinheadmz promag provoostenator ryanofsky sdaftuar S3RK stickies-v sipa sr_gi theStack TheCharlatan vasild
<josie> hi
<furszy> hi
<darosior> Hi
<instagibbs> hi
<TheCharlatan> hi
<hebasto> hi
<maxedw> hi
<laanwj> hii
<brunoerg> hi
<cfields> hi
<_aj_> hi
<achow101> There is one preproposed meeting topic this week. Any last minute ones to add?
<achow101> #topic package relay updates (glozow)
<dergoegge> hi
<pinheadmz> hi
<glozow> Nothing new this week - #28984 is still the priority. My main focus this week will be reviewing that. I think we are aiming for #29496 and #28984 for 28.0.
<glozow> On the p2p side, I just rebased #30111 which is the main PR to review.
<gribble> https://github.com/bitcoin/bitcoin/issues/28984 | Cluster size 2 package rbf by instagibbs · Pull Request #28984 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/29496 | policy: bump TX_MAX_STANDARD_VERSION to 3 by glozow · Pull Request #29496 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/28984 | Cluster size 2 package rbf by instagibbs · Pull Request #28984 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/30111 | locks: introduce mutex for tx download, flush rejection filters on UpdatedBlockTip by glozow · Pull Request #30111 · bitcoin/bitcoin · GitHub
<glozow> That's all from me
<willcl-ark> hi
<achow101> #topic cluster mempool updates (sdaftuar)
<sipa> Unsure if Suhas is around.
<sdaftuar> not too much from me this week either. i've rebased #28676 (on master) to try to get that code current again, and i'm planning to further get branches of that code built on sipa's cluster linearization branch (#30126) and the cluster size 2 package rbf PR
<gribble> https://github.com/bitcoin/bitcoin/issues/28676 | [WIP] Cluster mempool implementation by sdaftuar · Pull Request #28676 · bitcoin/bitcoin · GitHub
<sdaftuar> just to verify that everything works
<gribble> https://github.com/bitcoin/bitcoin/issues/30126 | Low-level cluster linearization code by sipa · Pull Request #30126 · bitcoin/bitcoin · GitHub
<instagibbs> 🙏 thanks
<sdaftuar> sipa -- anything to add on your PR? (that's the one to review right now)
<sipa> Not really, I've updated my low-level linearization code PR to include post-linearization and merging, which is all that I intend to implement at that "layer".
<sipa> Based on review, it may mean splitting some parts off.
<glozow> iiuc we're currently reviewing #30160 and #30161 to get those bits out of the way?
<gribble> https://github.com/bitcoin/bitcoin/issues/30160 | util: add BitSet by sipa · Pull Request #30160 · bitcoin/bitcoin · GitHub
<gribble> https://github.com/bitcoin/bitcoin/issues/30161 | util: add VecDeque by sipa · Pull Request #30161 · bitcoin/bitcoin · GitHub
<instagibbs> I think splitting will be necessary but haven't spent enough time on it to know how, focusing on those 2 prs first
<jon_atack> hi
<sipa> Indeed, those are direct dependencies, though I think it's reasonable to look at the algorithmic parts of #30126 already if you don't care that much about the data structures involved.
<gribble> https://github.com/bitcoin/bitcoin/issues/30126 | Low-level cluster linearization code by sipa · Pull Request #30126 · bitcoin/bitcoin · GitHub
<tdb3_> hi
<glozow> makes sense
<Murch[m]> hi
<sipa> My next plan is to start helping with integration into sdaftuar's PR so we can get the TxGraph abstraction to get an implementation using these algorithms
<sipa> But that won't change what's next for review.
<sipa> That's it from me, I think.
<achow101> #topic legacy wallet removal updates (achow101)
<abubakarsadiq> hi
<achow101> #26596 has been getting some review. It's still the priority pr.
<gribble> https://github.com/bitcoin/bitcoin/issues/26596 | wallet: Migrate legacy wallets to descriptor wallets without requiring BDB by achow101 · Pull Request #26596 · bitcoin/bitcoin · GitHub
<achow101> There's also one more item in the roadmap for 28.0 that isn't started yet: GUI: Migrate wallets that are not loaded
<achow101> will be looking at doing that soon, unless someone else wants to do it
<achow101> #topic Ad-hoc high priority for review
<achow101> Anything to add or remove from https://github.com/orgs/bitcoin/projects/1/views/4
<achow101> #topic vulnerability disclosure policy (darosior)
<darosior> Alright, so as you know we discussed better tracking and disclosing security bugs at the last two coredevs. About three months ago i also updated people during this meeting on Niklas and i writing about historical reports which were never disclosed. After the last coredev a few of us (dergoegge, sipa, _aj_, fanquake, achow101 and i) had a couple
<darosior> calls to discuss what could be a good disclosure policy for the project. We came up with a plan and a disclosure policy we are content starting with. To avoid a large wall of text here i've made it into a gist: https://gist.github.com/darosior/eb71638f20968f0dc896c4261a127be6.
freesprung51269 has quit [Ping timeout: 272 seconds]
<darosior> We'd like to seek feedback from everyone here. Keep in mind this is not set in stone, the policy may be improved or updated in the future. This is just a reasonable starting point which improves over the current situation. I can also give a TL;DR here to kick off discussions.
<sipa> i think a tl;dr is useful
<achow101> I guess more specific feedback can be also be left on the gist outside of the meeting?
<darosior> TL;DR:
<darosior> - disclose low severity bugs when a fix is released, after a pre-announcement period of 2 weeks.
<darosior> - the plan is to gradually phase-in this policy by first disclosing all the historical reports (up to version 0.21.x). Then we do one version per month. In July all which was fixed in 22.x. In August, 23.x. Etc..
<darosior> - disclose higher severity bugs once all affected versions are EOL, after a pre-announcement period of 2 weeks.
_andrewtoth_ has joined #bitcoin-core-dev
<darosior> Up until we run out of EOL releases to disclose security bugs for
<sdaftuar> Seems resonable to me on first glance -- thanks for working on this
<glozow> +1 thanks for working on this
<achow101> also the critical bugs that affect the entire network (e.g. inflation bug) will be handled on an ad-hoc basis
andrewtoth_ has quit [Remote host closed the connection]
<instagibbs> +1
<darosior> achow101: yes if people need more time to think about it feel free to comment on the gist. Since we've got no other topics i'm also fine waiting a couple minutes here for people to read through it.
<sipa> And just to highlight what the goal is here: we'd like to improve the public's perception about some of the work that is not always very visible, and the fact that there definitely are reasons to upgrade (even to backport releases)
<darosior> The next step would be to put up a blog post at bitcoincore.org with the historical bugs, and publish that on the ML at the same time as a request for feedback on our policy.
<darosior> Yes, and having a proper policy in place could also potentially incentivize security researchers to look at our stuff
<sipa> right
<josie> overall, looks great and glad to see this moving forward. regarding high severity disclosures, do we currently send announcements for releases reaching EOL? if we will be disclosing bugs 2 weeks after EOL, maybe we also make general announcements to remind people which releases will be EOL by the next release?
<josie> (maybe we already do this?)
<darosior> Also it's just useful to us as a group to learn from what past bugs were vulnerabilities, and how to try and prevent them moving forward
<achow101> josie: we don't currently. the plan would mean that we will
<pinheadmz> darosior great writeup, good plan and +1 on blog posts, looking forward to reading those myself!
<josie> cool, i think thats important, otherwise the 2 weeks after EOL seems a bit aggressive
<sipa> there is also a possibility of pre-announcements like openssl does (on day X we will be disclousing a bug affecting versions Y, of severity Z)
<darosior> pinheadmz: some are quite nice :)
<achow101> josie: essentially we will announce eol, and pre-announce that things will be disclosed in 2 weeks
<instagibbs> don't be afraid to extract maximum publicity for the series of posts/releases
<TheCharlatan> is there a reason for doing one version per month in this "catch-up" period?
<darosior> instagibbs: yeah i pinged the optech guys 👍
<darosior> TheCharlatan: lots of 22.0 still running
<darosior> Give a chance to people to upgrade
<achow101> TheCharlatan: we don't want to dump everything on everyone at once since a lot of people are still running eol software. phasing in gives people time to upgrade
<sipa> or more meta, give people some time to adapt their update processes in function of our process
<josie> achow101: maybe we are saying the same thing, but what im suggesting is "release X is out, also as a reminder Y and Z will be EOL in the next release (~ 6 months)" and then in 6 months "hey remember those EOL releases? we will disclose bugs in 2 weeks"
<achow101> josie: yeah, basically
<darosior> josie: yes
<jon_atack> lgtm. and good point josie. looking forward to reading these.
<darosior> Cool, thanks for the feedback. I guess that's it then.
<josie> big thanks to everyone who worked on this!
<achow101> Any other topics to discuss?
Emc99 has quit [Quit: Client closed]
Emc99 has joined #bitcoin-core-dev
<achow101> #endmeeting
_andrewtoth_ has quit [Remote host closed the connection]
_andrewtoth_ has joined #bitcoin-core-dev
freesprung51269 has joined #bitcoin-core-dev
Emc99 has quit [Client Quit]
twistedline_ has quit [Remote host closed the connection]
twistedline has joined #bitcoin-core-dev
preimage has joined #bitcoin-core-dev
bugs_ has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] pinheadmz opened pull request #30238: json-rpc 2.0 followups: docs, tests, cli (master...json2-followup) https://github.com/bitcoin/bitcoin/pull/30238
BrandonOdiwuor has quit [Ping timeout: 250 seconds]
marsupialSoup has joined #bitcoin-core-dev
salvatoshi has quit [Ping timeout: 264 seconds]
kevkevin has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 256 seconds]
SpellChecker_ has quit [Remote host closed the connection]
SpellChecker has joined #bitcoin-core-dev
_andrewtoth_ has quit [Ping timeout: 260 seconds]
SpellChecker has quit [Ping timeout: 260 seconds]
kevkevin has joined #bitcoin-core-dev
SpellChecker has joined #bitcoin-core-dev
kevkevin has quit [Ping timeout: 268 seconds]
zeropoint has joined #bitcoin-core-dev
Talkless has joined #bitcoin-core-dev
SpellChecker has quit [Ping timeout: 260 seconds]
BrandonOdiwuor has joined #bitcoin-core-dev
Guest60 has joined #bitcoin-core-dev
Guest60 has quit [Client Quit]
SpellChecker has joined #bitcoin-core-dev
luke-jr has quit [Remote host closed the connection]
luke-jr has joined #bitcoin-core-dev
marsupialSoup has quit [Ping timeout: 240 seconds]
Emc99 has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
jon_atack has quit [Ping timeout: 256 seconds]
Emc99 has quit [Quit: Client closed]
luke-jr_ has joined #bitcoin-core-dev
luke-jr has quit [Ping timeout: 268 seconds]
<bitcoin-git> [bitcoin] instagibbs opened pull request #30239: Ephemeral Anchors, take 2 (master...2024-03-general-ephemeral) https://github.com/bitcoin/bitcoin/pull/30239
<bitcoin-git> [bitcoin] instagibbs closed pull request #29001: Ephemeral Anchors (master...2023-12-ephemeral-anchors) https://github.com/bitcoin/bitcoin/pull/29001
marsupialSoup has joined #bitcoin-core-dev
puchka1101 has joined #bitcoin-core-dev
puchka has quit [Ping timeout: 252 seconds]
cfields has quit [Ping timeout: 256 seconds]
Guest13 has joined #bitcoin-core-dev
Guest13 has quit [Client Quit]
Guest27 has joined #bitcoin-core-dev
Guest27 has quit [Client Quit]
cfields has joined #bitcoin-core-dev
BrandonOdiwuor has quit [Quit: Client closed]
cfields has quit [Ping timeout: 240 seconds]
cfields has joined #bitcoin-core-dev
_andrewtoth_ has joined #bitcoin-core-dev
fundamentals has joined #bitcoin-core-dev
fundamentals has quit [Client Quit]
gribble has quit [Remote host closed the connection]
gribble has joined #bitcoin-core-dev
<cfields> sdaftuar / sipa: I've started working on a boost-less txmempool. So far I have a POC dumb impl that simply hard-codes everything and naively/inefficiently uses std containers. It works and passes all tests. I'm now looking into doing it properly with custom structures and a bit more template magic. Is #28676 sufficiently far along to give me an idea of what's going to change?
<gribble> https://github.com/bitcoin/bitcoin/issues/28676 | [WIP] Cluster mempool implementation by sdaftuar · Pull Request #28676 · bitcoin/bitcoin · GitHub
nicky has joined #bitcoin-core-dev
Talkless has quit [Remote host closed the connection]
<sipa> cfields: probably too early to say, but sdaftuar will know better
<sipa> we may want something multiindexy inside txgraph too
<sipa> ah, it already uses one, but the code there is likely to change a lot
abubakarsadiq has quit [Quit: Connection closed for inactivity]
<cfields> ok, I'll hold off on going any further
<sipa> is boost multiindex an issue, or something to get rid off, given that it's headers-only anyway?
<cfields> Imo it depends how much work it is. That's what I'm aiming to find out.
<sipa> fair enough
bugs_ has quit [Quit: Leaving]
luke-jr_ has quit [Remote host closed the connection]
luke-jr_ has joined #bitcoin-core-dev
marsupia1 has joined #bitcoin-core-dev
marsupialSoup has quit [Ping timeout: 268 seconds]
jonatack has quit [Ping timeout: 256 seconds]
abubakarsadiq has joined #bitcoin-core-dev
jonatack has joined #bitcoin-core-dev
kevkevin has joined #bitcoin-core-dev
marsupia1 has quit [Ping timeout: 246 seconds]
marsupialSoup has joined #bitcoin-core-dev
kevkevin has quit [Remote host closed the connection]
kevkevin has joined #bitcoin-core-dev
_andrewtoth_ has quit [Ping timeout: 260 seconds]
qxs has quit [Ping timeout: 260 seconds]
mudsip has joined #bitcoin-core-dev
qxs has joined #bitcoin-core-dev
Torr has joined #bitcoin-core-dev
mudsip has quit []
kevkevin has quit [Ping timeout: 264 seconds]
jon_atack has joined #bitcoin-core-dev
jonatack has quit [Ping timeout: 255 seconds]
grubles_ has quit [Quit: brb]
preimage has quit [Quit: WeeChat 4.2.2]
cygnet3 has quit [Quit: Leaving]
marsupialSoup has quit [Ping timeout: 264 seconds]
bitdex has joined #bitcoin-core-dev
<bitcoin-git> [bitcoin] achow101 pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/1040a1fc807e...4a020ca443ba
<bitcoin-git> bitcoin/master faf3cd6 MarcoFalke: test: Normalize struct.pack format
<bitcoin-git> bitcoin/master faf2a97 MarcoFalke: test: Use int.to_bytes over struct packing
<bitcoin-git> bitcoin/master fa826db MarcoFalke: scripted-diff: test: Use int.to_bytes over struct packing
<bitcoin-git> [bitcoin] achow101 merged pull request #29401: test: Remove struct.pack from almost all places (master...2402-nuke-struct-pack-) https://github.com/bitcoin/bitcoin/pull/29401
zeropoint has quit [Quit: leaving]
Guest13 has joined #bitcoin-core-dev
Guest13 has quit [Client Quit]