Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Newsletters: add 344 (2025-03-07) #2211

Merged
merged 3 commits into from
Mar 7, 2025

Conversation

harding
Copy link
Collaborator

@harding harding commented Mar 5, 2025

- **Disclosure of fixed LND vulnerability allowing theft:** Matt
Morehouse [posted][morehouse failback] to Delving Bitcoin to announce
the [responsible disclosure][topic responsible disclosures] of a
vulnerability that affected LND versions before 0.18. Upgarding to
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
vulnerability that affected LND versions before 0.18. Upgarding to
vulnerability that affected LND versions before 0.18. Upgrading to

update the specification.

- **Discussion about Bitcoin Core's priorities:** several blog posts
written by Antoine Poinsot about the future of the Bitcoin Core
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
written by Antoine Poinsot about the future of the Bitcoin Core
by Antoine Poinsot about the future of the Bitcoin Core

written by Antoine Poinsot about the future of the Bitcoin Core
project were linked in a [thread][poinsot pri] on Delving Bitcoin. In
the [first][poinsot pri1] blog post, Poinsot describes the benefits of
long-term goal setting and the costs of ad hoc decision making. In
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
long-term goal setting and the costs of ad hoc decision making. In
long-term goal setting and the costs of ad hoc decision-making. In

Comment on lines 48 to 49
tightly coupled. Many changes to one project would require also
making changes to the other projects. However, it would be a clear
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
tightly coupled. Many changes to one project would require also
making changes to the other projects. However, it would be a clear
tightly coupled. Many changes to one project would also require
changes to the other projects. However, it would be a clear

Comment on lines 49 to 56
making changes to the other projects. However, it would be a clear
win to move features that don't currently require a node into a
library or tool that can be maintained independently. He also
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
making changes to the other projects. However, it would be a clear
win to move features that don't currently require a node into a
library or tool that can be maintained independently. He also
making changes to the other projects. However,
moving features that don't currently require a node into a
library or tool that can be maintained independently would be a clear win. He also

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leaving as-is. The motivation for this change is unclear to me.

- **Trusted block templates:** a miner connects to a marketplace, selects
the bids it wants to include in a block, and asks the marketplace to
construct a block template. The marketplace responds with a block
header, coinbase transaction, and partial merkle branch that allow
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
header, coinbase transaction, and partial merkle branch that allow
header, coinbase transaction, and partial merkle branch, allowing

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keeping as-is. I think both constructions are correct and the existing construction avoids the use of a comma shortly after a comma-delimited list, making the sentence easier to skim.

transaction to the marketplace, which verifies the proof of work,
adds it to the block template, and
broadcasts the block. The marketplace might include a transaction
paying the miner in the block template or it might pay the miner
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
paying the miner in the block template or it might pay the miner
paying the miner in the block template, or it might pay the miner

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Keeping as-is. "It might pay" doesn't make sense here as an independent sentence, therefore it's not an independent clause, therefore this is a simple conjunction.

[TEE][] secure enclave, connect to marketplaces and select bids they
want to include in their blocks, and receive the transactions in
those bids encrypted to the TEE's enclave key. The block template
is constructed within the TEE and it provides the host operating
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
is constructed within the TEE and it provides the host operating
is constructed within the TEE, and provides the host operating

provides it to the TEE, which verifies it and returns the full
decrypted block template for the miner to add to the header and broadcast. Again, the block
template might include a transaction paying the miner from a UTXO
belonging to the marketplace or the marketplace might pay the miner
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
belonging to the marketplace or the marketplace might pay the miner
belonging to the marketplace, or the marketplace might pay the miner

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same comment as above: #2211 (comment)

belonging to the marketplace or the marketplace might pay the miner
later.

Both schemes would effectively require there to be multiple competing
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Both schemes would effectively require there to be multiple competing
Both schemes would effectively require multiple competing

brings the [BOLT12][] parameter setting in line with that of [BOLT11][].

- [Rust Bitcoin #4114][] lowers the minimum non-witness transaction size from 85
bytes to 62 bytes, aligning with Bitcoin Core’s policy (see Newsletter
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this "62" is a typo that should be "65". Will make that change.

@harding
Copy link
Collaborator Author

harding commented Mar 7, 2025

Made edits for all feedback, thanks @LarryRuane ! Added lede, updated releases/RCs, added topic entries. Skipped adding a MEV topic page but added it to #721 (comment) . Reviewed and made some small edits to @Gustavojfe section (thanks!).

Thanks everyone!

@bitschmidty bitschmidty force-pushed the 2025-03-07-newsletter branch from f77ab6b to 8bc7072 Compare March 7, 2025 12:35
@bitschmidty
Copy link
Contributor

Didnt have a chance to review fully this week, just a skim, but looks great and will have to come back.

Squashed

@bitschmidty bitschmidty merged commit cef300a into bitcoinops:master Mar 7, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants