Skip to content

fix: validate certificate results QC phase - #540

Open
andrewnguyen22 wants to merge 4 commits into
developmentfrom
fix/certificate-results-qc-phase
Open

fix: validate certificate results QC phase#540
andrewnguyen22 wants to merge 4 commits into
developmentfrom
fix/certificate-results-qc-phase

Conversation

@andrewnguyen22

@andrewnguyen22 andrewnguyen22 commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Changelog:

  • Reject certificate-result QCs unless they are precommit votes.
  • Recheck the phase before applying certificate results.
  • Require post-election proposer messages and their QCs to reference the same view.
  • Use the existing x/ namespace consistently for SMT writes, historical proofs, and compaction.
  • Harden store bounds, indexing, backups, derived stores, and transactions; reject mismatched transaction message names.

@andrewnguyen22
andrewnguyen22 marked this pull request as ready for review August 26, 2026 13:10
@andrewnguyen22
andrewnguyen22 marked this pull request as draft August 26, 2026 13:11
@andrewnguyen22
andrewnguyen22 marked this pull request as ready for review August 28, 2026 14:03
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