Skip to content

fix(malachite): gate proposal parts by validator peers#5536

Open
ukint-vs wants to merge 4 commits into
masterfrom
vs/malachite-validator-identities
Open

fix(malachite): gate proposal parts by validator peers#5536
ukint-vs wants to merge 4 commits into
masterfrom
vs/malachite-validator-identities

Conversation

@ukint-vs

Copy link
Copy Markdown
Member

Summary

  • Replaces the Malachite validator public-key table with validator identities: address -> { public_key, peer_id }.
  • Validates configured Malachite peer IDs at load/startup, including malformed IDs, duplicates, missing active validator identities, and local validator peer-id mismatches.
  • Builds and refreshes the active validator peer allowlist, then drops proposal parts from non-validator peers before stream buffering or pending DB writes.
  • Updates the example TOML files to document the validator-identities JSON shape.

Split out from #5514 so the bounded proposal-validation PR stays focused.

Tests

  • rtk rg "validator-pub-keys" ethexe/.ethexe.example.toml ethexe/.ethexe.example.local.toml
  • rtk cargo nextest run -p ethexe-cli validator_identities
  • rtk cargo nextest run -p ethexe-malachite-core
  • rtk cargo nextest run -p ethexe-malachite
  • rtk cargo clippy -p ethexe-cli -p ethexe-malachite-core -p ethexe-malachite --all-targets -- -D warnings
  • rtk git diff --check origin/master...HEAD

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

Gemini encountered an error creating the summary. You can try again by commenting /gemini summary.

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.

1 participant