BIP451: Dust UTXO Disposal Protocol#2150
Conversation
jonatack
left a comment
There was a problem hiding this comment.
Looks complete and correct at first glance.
4089a81 to
d65128a
Compare
murchandamus
left a comment
There was a problem hiding this comment.
Looks pretty good already! Thanks again for working on this so diligently before submitting it here.
|
Let’s call this BIP 451. Please add the number to the preamble, set the Assigned header to today, rename the BIP file, and add an entry to the README table. |
|
I pushed 54bccb1 with some small edits. I'll squash this and other commits once the review process is done. |
* Change title, abstract, motivation to focus on dust attack UTXOs * Simpify dust selection section * Add batching to address consolidation rules * Fix core version in privacy preservation * Fix table units
192d0ae to
54bccb1
Compare
|
Yeah, thanks, adding separate commits without squashing them during the review phase is preferred. |
f2d7074 to
26b76eb
Compare
26b76eb to
a9dcb9e
Compare
murchandamus
left a comment
There was a problem hiding this comment.
The Preamble updates and table entry look good. Thanks for the quick turn-around.
Given that you said that you are still mulling over some of the review comments, I’ll leave the “PR Author action required” tag for the time being. Please let me know when you’re ready for another review.
|
@murchandamus this one is ready for another look. In 686c1b0 I updated it to use only OP_RETURN "ash" and some other small edits. |
Discussion:
co-author: @harismuzaffer