Skip to content

params: reject chain config missing blobSchedule when Osaka is enabled - #1021

Open
ian0371 wants to merge 1 commit into
kaiachain:devfrom
ian0371:fix/require-blob-schedule
Open

params: reject chain config missing blobSchedule when Osaka is enabled#1021
ian0371 wants to merge 1 commit into
kaiachain:devfrom
ian0371:fix/require-blob-schedule

Conversation

@ian0371

@ian0371 ian0371 commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

A hand-written genesis that enables Osaka without a blobSchedule nil-dereferenced in ChainConfig.BlobConfig, killing an EN on a single eth_feeHistory call and panicking a CN at block-1 validation.

CheckConfigForkOrder now requires a valid Osaka blob schedule (with non-zero baseFeeUpdateFraction, which is a division denominator) so such a config is rejected at load, and BlobConfig is nil-safe as a second layer.

Types of changes

  • 🐛 Bug fix
  • ✨ Non-hardfork changes (node upgrade not required)
  • 💥 Hardfork / consensus-breaking changes
  • 🧪 Test improvements
  • 🧰 CI / build tool
  • ♻️ Chore / Refactor / Non-functional changes

Checklist

  • 📖 I have read the CONTRIBUTING GUIDELINES doc
  • 📝 I have signed in the PR comment I have read the CLA Document and I hereby sign the CLA in first time contribute after having read CLA
  • 🟢 Lint and unit tests pass locally with my changes ($ make test)

Related issues

Further comments

A hand-written genesis that enables Osaka without a blobSchedule
nil-dereferenced in ChainConfig.BlobConfig, killing an EN on a single
eth_feeHistory call and panicking a CN at block-1 validation.

CheckConfigForkOrder now requires a valid Osaka blob schedule (with
non-zero baseFeeUpdateFraction, which is a division denominator) so
such a config is rejected at load, and BlobConfig is nil-safe as a
second layer.
@ian0371 ian0371 self-assigned this Aug 7, 2026
@ian0371 ian0371 added the do not merge Do not merge just yet label Aug 7, 2026
@ian0371 ian0371 mentioned this pull request Aug 7, 2026
23 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Do not merge just yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants