Skip to content

header changes#805

Open
vbhattaccmu wants to merge 7 commits intoghali/infinity-dafrom
header-changes
Open

header changes#805
vbhattaccmu wants to merge 7 commits intoghali/infinity-dafrom
header-changes

Conversation

@vbhattaccmu
Copy link
Copy Markdown
Contributor

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

  • Feature
  • Bugfix
  • Refactor
  • Format
  • Documentation
  • Testing
  • Other:

Description

Related Issues

Testing Performed

Checklist

  • I have performed a self-review of my own code.
  • The tests pass successfully with cargo test.
  • The code was formatted with cargo fmt.
  • The code compiles with no new warnings with cargo build --release and cargo build --release --features runtime-benchmarks.
  • The code has no new warnings when using cargo clippy.
  • If this change affects documented features or needs new documentation, I have created a PR with a documentation update.

@vbhattaccmu vbhattaccmu marked this pull request as ready for review March 4, 2026 09:01
Copy link
Copy Markdown
Contributor

@ToufeeqP ToufeeqP left a comment

Choose a reason for hiding this comment

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

Where are we publishing the commitments now so that light-nodes can use them while validating the ev-proof & sampling?

)));
}

avail_blob::validation::validate_fri_proof(
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.

@vbhattaccmu Validators should still validate the fri_proof of all the blobs in a block.

Copy link
Copy Markdown
Contributor

@ToufeeqP ToufeeqP Mar 18, 2026

Choose a reason for hiding this comment

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

And we also need to validate the commitments root that the block authoring node generates & puts in the header

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.

3 participants