Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable taproot until we find a solution to prevent chain bloats #115

Merged
merged 2 commits into from
Apr 1, 2023

Conversation

SmartArray
Copy link

@SmartArray SmartArray commented Apr 1, 2023

This PR effectively disables taproot by setting it's activation date to year 2099.

We effectively disabled the activation by postponing it (instead of removing all taproot related code) so that we can easily reintroduce it in a future release when we have a fix ready to prevent the blockchain bloat that is happening on other chains right now. Additionally, this has the positive side effect that the functional tests regarding the activation of taproot will remain working.

Author's note

I am all for NFTs but it makes much more sense to store secure cryptographic hashes of your data instead of the raw data itself.

@SmartArray SmartArray force-pushed the fix/taproot-delay-activation branch from 58e371b to f2da74a Compare April 1, 2023 03:31
@ycagel ycagel requested review from JaredTate, ycagel, gto90 and j50ng April 1, 2023 13:43
Copy link

@JaredTate JaredTate left a comment

Choose a reason for hiding this comment

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

tACK. I also just added the same delay for Regtest and Testnet taproot activation code. I agree this first R/C should be delayed till we have more time for a proper solution.

Copy link
Member

@ycagel ycagel left a comment

Choose a reason for hiding this comment

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

cACK - Great work guys!

Copy link
Member

@gto90 gto90 left a comment

Choose a reason for hiding this comment

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

tACK

@gto90 gto90 merged commit 52875dc into develop Apr 1, 2023
@gto90 gto90 deleted the fix/taproot-delay-activation branch December 18, 2024 04:01
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