Skip to content

Backport uncapped datacarrier#78

Merged
ajtowns merged 3 commits intobitcoin-inquisition:29.xfrom
ajtowns:202507-inq29-datacarrier
Jul 11, 2025
Merged

Backport uncapped datacarrier#78
ajtowns merged 3 commits intobitcoin-inquisition:29.xfrom
ajtowns:202507-inq29-datacarrier

Conversation

@ajtowns
Copy link

@ajtowns ajtowns commented Jul 9, 2025

Cherry-picked from bitcoin#32406.

instagibbs and others added 3 commits July 4, 2025 20:08
Datacarrier output script sizes and output counts are now
uncapped by default.

To avoid introducing another startup argument, we modify the
OP_RETURN accounting to "budget" the spk sizes.

If a user has set a custom default, this results in that
budget being spent over the sum of all OP_RETURN outputs'
scripts in the transaction, no longer capping the number
of OP_RETURN outputs themselves. This should allow a
superset of current behavior while respecting the passed
argument in terms of total arbitrary data storage.

Co-authored-by: Anthony Towns <aj@erisian.com.au>
Credit: Sjors Provoost and Antoine Poinsot
@ajtowns ajtowns added this to the 29.x milestone Jul 9, 2025
Copy link

@darosior darosior left a comment

Choose a reason for hiding this comment

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

ACK 79e42ac

@ajtowns ajtowns merged commit e924306 into bitcoin-inquisition:29.x Jul 11, 2025
18 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