Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

Add hints into proofs to help validation #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

afilini
Copy link
Contributor

@afilini afilini commented Oct 7, 2018

No description provided.

@afilini afilini requested a review from isghe October 7, 2018 12:30
@afilini
Copy link
Contributor Author

afilini commented Oct 31, 2018

Should we make this mandatory?

@afilini
Copy link
Contributor Author

afilini commented Mar 25, 2019

After a few months, an update: as suggested by @gabridome we should also add the confirmation blockheight (when known) to allow nodes without txindex to easily validate these proofs.

I would personally make the txid mandatory, while the blockheight would be optional for obvious reasons.

Also, a versioning system should be added to make it easier to deserialize proofs, to avoid the "try-catch" method where you assume some fields are present and fallback to a different strategy if they are not.

@gabridome
Copy link

@afilini or the blockhash directly.
It depends what we want to preserve: cpu or storage/bandwidth.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants