bips icon indicating copy to clipboard operation
bips copied to clipboard

BIP153: SENDTEMPLATE

Open ajtowns opened this issue 4 months ago • 14 comments

Adds a BIP for SENDTEMPLATE, GETTEMPLATE, TEMPLATE p2p messages. Discussion links:

https://delvingbitcoin.org/t/sharing-block-templates/1906

https://gnusha.org/pi/bitcoindev/[email protected]/T/#u

ajtowns avatar Aug 18 '25 01:08 ajtowns

Post to bitcoindev group hasn't made it through

fixed, published

jonatack avatar Aug 18 '25 16:08 jonatack

Updated with post-history in metadata and some rationale

ajtowns avatar Aug 27 '25 04:08 ajtowns

Time to request a bip number then I guess?

ajtowns avatar Sep 01 '25 08:09 ajtowns

Let’s call this BIP 153 then!

murchandamus avatar Sep 04 '25 20:09 murchandamus

@ajtowns: What’s your status on this one? Are you still planning changes or waiting for some people to finish their review of this PR?

murchandamus avatar Sep 16 '25 21:09 murchandamus

I think this is fine to merge as draft, further review/changes can happen in that state as I understand it.

ajtowns avatar Sep 16 '25 22:09 ajtowns

I may have overlooked it, but at first glance it looks like the draft is missing important specification on how to implement the goals set forth in the Motivation section. For instance, which peers to request templates from, when/how often, and how the template data is to be used.

I see those as implementation quality issues, which would be added in the "recommendations" section, once it's more clear what those recommendations should be.

ajtowns avatar Sep 18 '25 03:09 ajtowns

I don't see how this doesn't have major privacy leaks, similar to the mempool message

luke-jr avatar Sep 18 '25 22:09 luke-jr

Could the editors please clarify which of these concerns are blockers for merging this BIP in "Draft" status?

ajtowns avatar Sep 26 '25 15:09 ajtowns

Could the editors please clarify which of these concerns are blockers for merging this BIP in "Draft" status?

I don’t perceive the stated concerns as blockers, but agree that this document should be enhanced in the future with further discussion of privacy implications. I’m gonna wait for a few days to allow time for others to comment.

murchandamus avatar Sep 29 '25 23:09 murchandamus

that this document should be enhanced in the future with further discussion of privacy implications.

That's what the "TBA" in the otherwise empty recommendations section is for (as well as efficiency implications and potentially other things)

ajtowns avatar Sep 30 '25 03:09 ajtowns

Could the editors please clarify which of these concerns are blockers for merging this BIP in "Draft" status?

I've been told some editors are privately blocking merge of this PR despite being unwilling to identify any issues as blockers publicly. I don't think that's appropriate, or in accordance with the BIP 2 process, so I'm instead requesting this PR be merged now as-is, publishing the BIP as a draft so it can be easily referenced for further discussion and exploration.

ajtowns avatar Oct 02 '25 02:10 ajtowns

My understanding is that some fingerprinting concerns and privacy concerns have been raised here and in the Bitcoin Core pull request where this feature is being explored. According to the BIP Process, raised concerns should be addressed in the document. I expect that in the context of the feature being proposed, the frequency at which and the context in which it is used is seen as relevant information to weigh in on the utility, viability, and privacy impact. Therefore it would be helpful for readers and reviewers, if there were at least an initial sketch how often and when it would be used.

murchandamus avatar Oct 02 '25 03:10 murchandamus

This is now published as BIN25-2 so I'll be using that repo to refine the document while it's in draft.

ajtowns avatar Oct 09 '25 06:10 ajtowns