ElementsProject / elements

Open Source implementation of advanced blockchain features extending the Bitcoin protocol
MIT License
1.04k stars 372 forks source link

destroyamount "too easy to use"? #716

Open instagibbs opened 4 years ago

instagibbs commented 4 years ago

Someone decided it was a good idea to "test" it out by burning some L-BTC, and was surprised it worked(?).

Though in the Liquid case that just donates to the watchmen, maybe make this harder.

Two-phase RPC that must be called to turn on "I like pain" mode?

apoelstra commented 3 years ago

Why do we even have an RPC for this?

stevenroose commented 3 years ago

Probably because people issue test tokens. I'd be ok to disallow destroyamount use with policyAsset.

On Sat, Nov 28, 2020 at 1:43 AM Andrew Poelstra notifications@github.com wrote:

Why do we even have an RPC for this?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ElementsProject/elements/issues/716#issuecomment-735025729, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGQLXE5RP7HN6BAGGDHFSDSSBIVTANCNFSM4IX7Z7NA .