Skip to content

[P2] Verify header blob for blocks with transactions too #47

Closed

Description

@auricom

Priority: P2 (feature / coverage) · Effort: S (after the DA cache issue)

Problem

pkg/exporters/verifier/verifier.go:223-226: blocks with transactions only verify the data namespace. Their header blobs are never checked, so header-namespace submission failures on those blocks go unnoticed.

Proposed fix

For blocks with transactions, verify both header and data, and record each outcome under its own type label.

Pros / Cons

  • ✅ Full coverage of what the tool claims to verify.
  • ⚠️ About 2× Celestia calls; do this after the DA-height cache issue, which makes it nearly free. submission_*{type="header"} counts will jump, so mention it in the release notes.

Activity

  1. auricom commented on Oct 20, 2026

    @auricom
    CollaboratorAuthor

    Unblocked: #40 (the Celestia per-DA-height cache) shipped in #58. Note that the header path uses Blob.Get by commitment, which differs per block and is not cached. Verifying headers for tx blocks therefore costs one extra Celestia RPC per tx block, plus one ev-node call already done today. The DA timestamp lookup is cached.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions