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

Implement full node with N-block confirmation delay #237

Closed
nathanielhourt opened this issue Aug 17, 2015 · 2 comments
Closed

Implement full node with N-block confirmation delay #237

nathanielhourt opened this issue Aug 17, 2015 · 2 comments
Assignees

Comments

@nathanielhourt
Copy link
Contributor

Pursuant to #234, option (3), implement an option on fork_database which prevents the pushing of blocks which have not yet received N confirmations. This option must be exposed in the config file, as well as being specifiable from the testing infrastructure.

@theoreticalbts
Copy link
Contributor

In order to implement (3), you need some sort of header validation on un-pushed blocks. If you don't do any header validation, you must ensure you only connect to node(s) that do, i.e. you will be using approach (2).

The witness scheduler algorithm gives you half a round of witnesses.

@nathanielhourt
Copy link
Contributor Author

Good point. And due to the inability to reliably determine a witness's key in future blocks, without doing full validation (because it may have been changed in the unprocessed blocks) we cannot do proper header validation.

Given this issue, rather than substantially rework consensus protocol, I'll be implementing approach (2) instead.

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

No branches or pull requests

2 participants