Skip to content

Newsletters: add 285 (2024-01-17)#1468

Merged
bitschmidty merged 2 commits intobitcoinops:masterfrom
harding:2023-01-17-newsletter
Jan 17, 2024
Merged

Newsletters: add 285 (2024-01-17)#1468
bitschmidty merged 2 commits intobitcoinops:masterfrom
harding:2023-01-17-newsletter

Conversation

@harding
Copy link
Collaborator

@harding harding commented Jan 14, 2024

Includes Ark topic that was previously in #1452

The new vulnerability was discovered by Morehouse's following up on
his prior work on fake funding, which he also responsibly disclosed
(see [Newsletter #266][news266 lnbugs]). When re-testing nodes that
had implemented fixes for fake funding, was able to trigger a [race
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
had implemented fixes for fake funding, was able to trigger a [race
had implemented fixes for fake funding, he was able to trigger a [race

In the thread, Black and others describe some of the protocols that
would be enabled by this combination of consensus changes:
[LN-Symmetry][topic eltoo] (eltoo), [Ark][topic ark]-style
[joinpools][topic joinpools], reduced-signatures [DLCs][topic dlc],
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe?

Suggested change
[joinpools][topic joinpools], reduced-signatures [DLCs][topic dlc],
[joinpools][topic joinpools], reduced-signature [DLCs][topic dlc],

that combines previous proposals for [OP_CHECKTEMPLATEVERIFY][topic
op_checktemplateverify] (CTV) and [OP_CHECKSIGFROMSTACK][topic
op_checksigfromstack] (CSFS) with a new proposal for an
`OP_INTERNALKEY` that places the taproot internal key on the stack.
Copy link
Contributor

Choose a reason for hiding this comment

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

We have a good explanation of taproot internal keys here: https://bitcoinops.org/en/newsletters/2019/05/14/#complex-spending-with-taproot

Perhaps that or another resource would be helpful for reader

attention:

- *CPFP carve out needs to be removed:* the [CPFP carve out][topic
cpfp carve out] mempool policy added to Bitcoin Core in FIXME:year
Copy link
Contributor

Choose a reason for hiding this comment

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

out unless we restrict the relationships allowed between
transactions relayed on the network far beyond the current
restrictions. A cluster with multiple carve outs could
significantly exceed its limits, at which point we'd need to
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe reword to take out "we’d"

Comment on lines +74 to +75
proposal, such as how to encode the integer value and what
[taproot][topic taproot] upgrade feature to use.
Copy link
Contributor

Choose a reason for hiding this comment

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

Could use a little more detail:

Suggested change
proposal, such as how to encode the integer value and what
[taproot][topic taproot] upgrade feature to use.
proposal such as what integer encoding to use and whether
creating a new set of arithmetic opcodes is preferred to
upgrading existing ones.

cluster mempool is [v3 transaction relay][topic v3 transaction
relay], which would allow regular users of v1 and v2 transactions
to continuing using them in all the historically typical ways, but
also allow the users of contract protocols like LN to opt-in to v3
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
also allow the users of contract protocols like LN to opt-in to v3
also allow the users of contract protocols like LN to opt in to v3

Comment on lines +141 to +142
that contain more than 0 satoshis. An ephemeral anchor pays an output
script equivalent to `OP_TRUE`, allowing anyone to spend it.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
that contain more than 0 satoshis. An ephemeral anchor pays an output
script equivalent to `OP_TRUE`, allowing anyone to spend it.
that contain more than 0 satoshis. An ephemeral anchor pays to an output
script equivalent to `OP_TRUE`, allowing anyone to spend it.

Personally I'd slightly rewrite this:

Suggested change
that contain more than 0 satoshis. An ephemeral anchor pays an output
script equivalent to `OP_TRUE`, allowing anyone to spend it.
that contain more than 0 satoshis. An ephemeral anchor pays to a
standardized output script that anyone can spend.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Nice improvement, thanks!

A proposed alternative is to put the trimmed HTLC amounts into the
value of the ephemeral anchor outputs. That way they incentivize
confirming both the commitment transaction and a spend of the
ephemeral anchor output. In his post, Sander's analyzes this
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ephemeral anchor output. In his post, Sander's analyzes this
ephemeral anchor output. In his post, Sanders analyzes this

Copy link
Collaborator

@murchandamus murchandamus left a comment

Choose a reason for hiding this comment

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

Very cool newsletter so far.

[joinpools][topic joinpools], reduced-signatures [DLCs][topic dlc],
and [vaults][topic vaults] without presigned transactions, among
other described benefits of the underlying proposals, such as
CTV-style congestion control and CSFS-style signature delegation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ugh, congestion control is such an absurd application, I really don’t understand why people keep bringing that up.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's still the first thing described in BIP119:

This BIP proposes a new opcode, OP_CHECKTEMPLATEVERIFY, to be activated as a change to the semantics of OP_NOP4.

The new opcode has applications for transaction congestion control and [...]

(But I also tend to think that it's impractical.)

projects. Please consider upgrading to new releases or helping to test
release candidates.*

- [LND 0.0.119][] in a new release of this library for building
Copy link
Contributor

@vostrnad vostrnad Jan 16, 2024

Choose a reason for hiding this comment

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

Suggested change
- [LND 0.0.119][] in a new release of this library for building
- [LDK 0.0.119][] is a new release of this library for building

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thank you @vostrnad! I just realized I missed the correction from LND to LDK.

projects. Please consider upgrading to new releases or helping to test
release candidates.*

- [LND 0.0.119][] is a new release of this library for building
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- [LND 0.0.119][] is a new release of this library for building
- [LDK 0.0.119][] is a new release of this library for building

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oof, thank you @azuchi !

[news269 rpc]: /en/newsletters/2023/09/20/#bitcoin-core-28448
[news85 stuck]: /en/newsletters/2020/02/19/#c-lightning-3500
[news89 stuck]: /en/newsletters/2020/03/18/#eclair-1319
[lnd 0.0.119]: https://github.com/lightningdevkit/rust-lightning/releases/tag/v0.0.119
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[lnd 0.0.119]: https://github.com/lightningdevkit/rust-lightning/releases/tag/v0.0.119
[ldk 0.0.119]: https://github.com/lightningdevkit/rust-lightning/releases/tag/v0.0.119

@harding
Copy link
Collaborator Author

harding commented Jan 17, 2024

Made all edits (thanks @bitschmidty @vostrnad @murchandamus @azuchi !), added lede and releases, reviewed @adamjonas contribution (thanks!), and added topic entries. Thanks everyone!

@bitschmidty bitschmidty force-pushed the 2023-01-17-newsletter branch from d08e42b to 125bc7b Compare January 17, 2024 11:48
@bitschmidty bitschmidty merged commit 36eec45 into bitcoinops:master Jan 17, 2024
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.

6 participants