Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add LightClientHeader wrapper #4481

Merged
merged 3 commits into from
Jan 13, 2023
Merged

Conversation

etan-status
Copy link
Contributor

The light client data structures were changed to accommodate additional fields in future forks (e.g., to also hold execution data).

There is a minor change to the JSON serialization, where the header properties are now nested inside a LightClientHeader. The SSZ serialization remains compatible.

See ethereum/consensus-specs#3190 and ethereum/beacon-APIs#287

@github-actions
Copy link

github-actions bot commented Jan 10, 2023

Unit Test Results

         9 files  +       3    1 050 suites  +350   31m 53s ⏱️ + 11m 56s
  3 502 tests +   254    3 265 ✔️ +     40  237 💤 +214  0 ±0 
15 194 runs  +5 062  14 929 ✔️ +4 843  265 💤 +219  0 ±0 

Results for commit baffdcd. ± Comparison against base commit 7e27693.

♻️ This comment has been updated with latest results.

@etan-status etan-status marked this pull request as ready for review January 12, 2023 18:18
@etan-status etan-status force-pushed the dev/etan/lc-headerwrapper branch from d8191dd to ade4542 Compare January 12, 2023 20:06
The light client data structures were changed to accommodate additional
fields in future forks (e.g., to also hold execution data).

There is a minor change to the JSON serialization, where the `header`
properties are now nested inside a `LightClientHeader`.
The SSZ serialization remains compatible.

See ethereum/consensus-specs#3190
and ethereum/beacon-APIs#287
@etan-status etan-status force-pushed the dev/etan/lc-headerwrapper branch from ade4542 to d33d6e7 Compare January 12, 2023 20:12
@etan-status
Copy link
Contributor Author

REST endpoint for testing

http://127.0.0.1:5052/eth/v1/events?topics=light_client_optimistic_update

@etan-status etan-status merged commit 2324136 into unstable Jan 13, 2023
@etan-status etan-status deleted the dev/etan/lc-headerwrapper branch January 13, 2023 15:46
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.

2 participants