Flag when blocks have had their scripts checked instead of skipped #19790
Conversation
|
Hmm, one shortcoming of this approach, is that it doesn't differentiate between "verified with ruleset X" from "verified with ruleset Y" as would occur when you upgrade for a softfork after activation. This is still better than nothing at all, but I wonder if we should add logic for the latter? Perhaps just a "ruleset version" in the db, which we can check and clear the flag on blocks appropriately? |
I think I'm a concept ACK on adding something to track which validation rules were applied when validating each block, so that after softfork activation we could efficiently revalidate the blocks that weren't validated with all the rules. I haven't given a lot of thought to how much work it is to do this though. |
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
|
🐙 This pull request conflicts with the target branch and needs rebase. Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft". |
|
Think this is better off being redone softfork-aware somehow. Closing for now. |
We raise blocks to
BLOCK_VALID_SCRIPTSregardless of whether we validate scripts or skip them.A more obvious change might be to demote scripts-skipped to
BLOCK_VALID_CHAINand allow this status as an ancestor ofBLOCK_VALID_SCRIPTS, but that would have compatibility issues at least.Instead, just begin setting an additional flag set when we don't skip script checks.
This could be useful for later going back and verifying scripts of skipped blocks post-sync, but it's not clear to me yet if that's useful with assumeutxo around the corner(?). Regardless, it seems like a good idea to save this detail in case it proves useful.