namecoin-core icon indicating copy to clipboard operation
namecoin-core copied to clipboard

Make dust spam unspendable

Open indolering opened this issue 11 years ago • 13 comments

Removing all 1 Satoshi outputs before block 50k would remove the majority of the dust-spam from the system.

indolering avatar Jul 17 '14 01:07 indolering

Yes. This is my next project, and maybe I can submit a patch "really soon". ;) We can then discuss the details (if my selection of excluded outputs is ok, when to softfork, and all that).

domob1812 avatar Jul 17 '14 05:07 domob1812

See https://github.com/namecoin/namecoin/pull/140.

domob1812 avatar Jul 18 '14 10:07 domob1812

Well, then just close this.

indolering avatar Jul 18 '14 17:07 indolering

Or not because Bounty Source won't do a bounty on a PR.

indolering avatar Jul 18 '14 17:07 indolering

@indolering Are the Bounty Source issues resolved? I would like to close this issue.

phelixbtc avatar Sep 14 '14 18:09 phelixbtc

@phelixbtc This should not be closed until the softfork is merged. (As far as I know it is not merged yet.)

JeremyRand avatar Sep 14 '14 22:09 JeremyRand

@JeremyRand There is a standing PR for this issue. Shouldn't we close this and move discussion there?

@phelixbtc Yes, the bounties were paid via the forum and we no longer need this as a placeholder.

indolering avatar Sep 15 '14 00:09 indolering

Closing as of Indolering's post.

phelixbtc avatar Sep 15 '14 10:09 phelixbtc

@phelixbtc @indolering It is my opinion that issues should not be closed until they are resolved in the master branch. This is consistent with standard policy for open-source projects, and also is standard policy for BountySource. BountySource allows bounties to be claimed as soon as an issue is closed, and if end users find that the issue is not fixed in master, they may choose to dispute the bounty, which is bad for everyone. As such, I'm re-opening this issue until the PR is merged. Furthermore, the PR is not in a mergeable state right now; the block depth has not been decided yet. That should be discussed in this thread, and closing the issue makes that more difficult. If there are objections, please give me time to respond -- the issue was closed only 10 hours after @indolering suggested doing so; I was not given adequate time to respond. Thanks.

JeremyRand avatar Sep 15 '14 19:09 JeremyRand

@JeremyRand Sorry for closing it so quick. IMHO the PR has "replaced" this issue. That's not strictly correct but I am confident that we will resolve the issue via the PR. The only reason for closing this issue was to keep the issue list lean and mean. If you prefer to keep the issue open it's fine with me. Edit: To explain: I had thought you had missed the PR.

phelixbtc avatar Sep 15 '14 20:09 phelixbtc

@JeremyRand there are no standing bounties at BountySource for this issue, the bounty was directly to Domob in a private exchange.

Sorry for being too hasty about suggesting to close it, I meant to defer to Jeremy.

indolering avatar Sep 15 '14 20:09 indolering

@domob1812 Can we roll the proposed fix (remove all 1-swartz outputs created between block heights 39k and 41k from the UTXO set) into the Taproot softfork event? Cyphrs is interested in this (they want to decrease the storage requirements of running a pruned node), and may be able to fund the work if needed.

JeremyRand avatar Mar 25 '22 09:03 JeremyRand

What would this actually entail? Just adding a rule to disallow spending those outputs is relatively easy of course. But if we want to reduce storage for a pruned node, we need to a) actually remove all of them from the UTXO set, and b) somehow do so already while syncing (not only at the fork block).

Of course that's still very possible to develop, but would be a lot more effort. Do we know what practical effect that would have in terms of reduced requirements? What new usecases would that enable?

domob1812 avatar Mar 28 '22 05:03 domob1812