Skip to content

Update still uses client_state.latest_slot rather than header.trusted_slot #671

@srdtrk

Description

@srdtrk

Summary of Bug

With the addition of historical updates in #503, verify client now uses header.trusted_slot to pick the consensus state to use for verify the header itself. However, update_client still uses client_state.latest_slot.

We can also instead add the requirement that latest_slot is in the same sync committee period trusted_slot, in which case, it wouldn't matter which one we used.

Version

main


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
  • Estimate provided

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingethereum-light-clientIssues related to the 08-wasm ethereum light client

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions