Skip to content
This repository was archived by the owner on Aug 3, 2026. It is now read-only.

fix: validate Shasta grandparent base fee - #696

Merged
smtmfft merged 3 commits into
hotfix/hotfix-based-on-1.16.1from
fix/shasta-grandparent-basefee-hotfix
Jun 26, 2026
Merged

fix: validate Shasta grandparent base fee#696
smtmfft merged 3 commits into
hotfix/hotfix-based-on-1.16.1from
fix/shasta-grandparent-basefee-hotfix

Conversation

@smtmfft

@smtmfft smtmfft commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • require the L2 grandparent header for non-genesis Shasta base-fee validation
  • bind the grandparent header to the parent via block number and parent hash before using its timestamp
  • run Shasta base-fee validation after manifest selection so invalid/default manifests cannot skip it
  • include dev fork env values in SGX manifests so devnet/mainnet builds use the same image measurement path
  • bump gaiko to 47944ca (taikoxyz/gaiko#31) to remove committed enclave signing material

Test Plan

  • cargo fmt -p raiko-lib -- --check
  • cargo test -p raiko-lib -- --nocapture
  • git diff --check -- lib/src/utils/shasta.rs lib/src/utils/shasta_rules.rs
  • git diff --check origin/hotfix/hotfix-based-on-1.16.1...HEAD
  • checked gaiko@47944ca does not contain docker/enclave-key.pem or PEM private-key material

@johntaiko
johntaiko requested a review from Copilot June 26, 2026 08:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@smtmfft
smtmfft merged commit 401097a into hotfix/hotfix-based-on-1.16.1 Jun 26, 2026
16 of 18 checks passed
@smtmfft
smtmfft deleted the fix/shasta-grandparent-basefee-hotfix branch June 26, 2026 09:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants