Consensus: enable COINBASE_FLAGS on PoS - #96
Open
naomi-mcrn wants to merge 1 commit into
Open
Conversation
If COINBASE_FLAGS is not empty, add it to coinbase to identify minter.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
COINBASE_FLAGSis defined at validation.cpp L254.Miner(minter) can set any data organization name that identify them. (example below)
But currently it is not available on PoS phase.
Because it is added to coinbase at
IncrementExtraNoncethat not called on PoS.This change enable
COINBASE_FLAGSon PoS phase.If miner didn't set data, nothing changes.
Otherwise, extra stuff is added to coinbase.
Block signature will NOT CORRUPT by this change because it is last part of coinbase scriptSig.
It was already verified on mainnet (e.g. https://insight.xpchain.io/tx/a96d635bf0ae90442ff566e7124d3a1adcf9771c72b39c83b069a926186c276c, created by my custom wallet).