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

feat(base_layer): basic checkpoint validation #4293

Conversation

mrnaveira
Copy link
Contributor

Description

  • Removed unnecessary validation about required contract definition in checkpoints
  • Added checkpoint validation of existing prior contract constitution
  • Added checkpoint validation of checkpoint signer public keys being included in the constitution committee
  • For convenience, implemented IntoIterator trait for references of CommitteeSignatures

Motivation and Context

The base layer must perform some basic validations on checkpoints:

  • The constitution must have been published
  • The checkpoint signers public keys must be included in the constitution committee

How Has This Been Tested?

The new unit tests for the new validations pass

@aviator-app aviator-app bot merged commit 045997a into tari-project:development Jul 8, 2022
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.

2 participants