Skip to content
This repository has been archived by the owner on Jun 24, 2024. It is now read-only.

allow rollup client update with multiple DA header update #174

Open
rnbguy opened this issue Jun 6, 2024 · 0 comments
Open

allow rollup client update with multiple DA header update #174

rnbguy opened this issue Jun 6, 2024 · 0 comments

Comments

@rnbguy
Copy link
Member

rnbguy commented Jun 6, 2024

Currently, the sovereign header has a single update header from DA.

pub struct Header<H> {
pub aggregated_proof: AggregatedProof,
pub da_header: H,
}

But it may happen, the DA header needs multiple updates to reach to the latest height. Currently, this requires creating multiple sovereign headers - which needs redundant queries to rollup endpoint for the rollup state roots.

This can be avoided by passing the required (possibly, multiple) DA headers in a single sovereign header.

pub struct Header<H> {
    pub aggregated_proof: AggregatedProof,
    pub da_headers: Vec<H>,
}
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: 📥 To Do
Development

No branches or pull requests

1 participant