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

broadcast optimistic light client updates #3499

Merged
merged 1 commit into from
Mar 17, 2022

Conversation

etan-status
Copy link
Contributor

After proposing a new block, broadcasts a OptimisticLightClientUpdate.
Works for both locally proposed blocks as well as VC submitted ones.

@etan-status etan-status requested a review from arnetheduck March 14, 2022 16:51
@github-actions
Copy link

github-actions bot commented Mar 14, 2022

Unit Test Results

     12 files     833 suites   37m 44s ⏱️
1 682 tests 1 633 ✔️   49 💤 0
9 807 runs  9 691 ✔️ 116 💤 0

Results for commit 4571740.

♻️ This comment has been updated with latest results.

@etan-status etan-status force-pushed the dev/etan/lc-broadcast branch from 3b350d6 to ad2a56e Compare March 15, 2022 14:19
@etan-status
Copy link
Contributor Author

  • Added comment with rationale for delaying OptimisticLightClientUpdate and spec reference.
  • Rename optimistic_light_client_update_deadline > optimistic_light_client_update_time to denote the fixed time at which the OptimisticLightClientUpdate is sent.

After proposing a new block, broadcasts a `OptimisticLightClientUpdate`.
Works for both locally proposed blocks as well as VC submitted ones.
@etan-status
Copy link
Contributor Author

  • Assert existing parent reference for newly produced block.

@etan-status etan-status force-pushed the dev/etan/lc-broadcast branch from ad2a56e to 4571740 Compare March 17, 2022 13:43
@etan-status etan-status merged commit 9f8894f into unstable Mar 17, 2022
@etan-status etan-status deleted the dev/etan/lc-broadcast branch March 17, 2022 20:11
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