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

As validator, check keys per epoch #3077

Merged
merged 1 commit into from
Feb 5, 2025
Merged

As validator, check keys per epoch #3077

merged 1 commit into from
Feb 5, 2025

Conversation

nibhar
Copy link
Member

@nibhar nibhar commented Nov 16, 2024

Since keys can change, this PR adds a basic check to the validator which makes sure its keys are consistent with the ones the election block expects. Similar checks are introduced to signing Level updates and Micro blocks such that a validator will not produce blocks if the keys do not match and not sign any LevelUpdates.

@nibhar nibhar force-pushed the nibhar/check-keys branch 4 times, most recently from c016f48 to cf8da80 Compare November 19, 2024 21:02
@Eligioo
Copy link
Member

Eligioo commented Jan 9, 2025

@nibhar can this PR get rebased?

@hrxi hrxi force-pushed the nibhar/check-keys branch from cf8da80 to 61b07ce Compare February 5, 2025 14:41
@hrxi hrxi force-pushed the nibhar/check-keys branch from 61b07ce to 2057b53 Compare February 5, 2025 14:42
@hrxi hrxi changed the base branch from albatross to iicruz/remove_live_sync_no_gaps_invariant2 February 5, 2025 14:43
Base automatically changed from iicruz/remove_live_sync_no_gaps_invariant2 to albatross February 5, 2025 15:19
@hrxi hrxi merged commit 2057b53 into albatross Feb 5, 2025
8 checks passed
@hrxi hrxi deleted the nibhar/check-keys branch February 5, 2025 15:24
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.

4 participants