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

Fix block verbosedata population… #2275

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

TMKCodes
Copy link

@TMKCodes TMKCodes commented May 5, 2024

Fix block verbosedata population from skipping transaction verbosedata population when BlockInfo errorenously reports block as Header only, but domainBlock is still fount with GetBlockEvenIfHeaderOnly.

Basically this affects archival servers. Once archival server moves on pruning point it loses verboseData from transactions as such transaction id is lost unless repopulated from the block data.

…a population when BlockInfo errorenously reports block as Header only, but domainBlock is still fount with GetBlockEvenIfHeaderOnly.
@michaelsutton michaelsutton changed the base branch from master to dev May 9, 2024 09:59
@bobpinella
Copy link

We are currently running an archive node and need this fix badly - will this PR be incorporated into the next version anytime soon?

@bobpinella
Copy link

bobpinella commented Sep 18, 2024

I just noticed that this version of kaspad has been deprecated. We have confirmed that this is not an issue with the Rust version.

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.

3 participants