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

feat(v2-eos-cores): <- allow those to work with multi incremerkles #16

Merged
merged 40 commits into from
Feb 26, 2024

Conversation

gskapka
Copy link
Contributor

@gskapka gskapka commented Jan 31, 2024

...per title.

The eos cores now store up to X previous incremerkles, allowing for submission material to contains proofs that reference any of those rather than just the chain tip.

This is to help mitigate the race condition that sometimes occurs between the two nodes used for eos cores, one for getting blocks and the other for actions.

If the action is slow to appear on the end point, occasionally the core's incremerkle can have been updated in the meanwhile, moving it beyond the block the action (when it appears) was dispated in. This leaves the submission stuck.

Now however, the cores when asked about their latest block numbers, return an array of options that the eos-syncer may chose from in order to correctly proof an action w/r/t to a block the core knows about.

gskapka added 28 commits January 4, 2024 10:40
@gskapka gskapka force-pushed the multi-incremerkles branch from 0a846e0 to b6ada32 Compare January 31, 2024 17:37
@gskapka gskapka force-pushed the multi-incremerkles branch from b6ada32 to 235d9a3 Compare January 31, 2024 17:48
@gskapka gskapka self-assigned this Jan 31, 2024
@gskapka gskapka requested a review from ubordignon January 31, 2024 18:04
…ng protection works w/ multi-incremerkle based submission
@gskapka
Copy link
Contributor Author

gskapka commented Feb 14, 2024

Awaiting testing on prod before merging.

ubordignon
ubordignon previously approved these changes Feb 14, 2024
Copy link
Contributor

@ubordignon ubordignon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@gskapka gskapka requested a review from ubordignon February 19, 2024 12:06
Copy link
Contributor

@ubordignon ubordignon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@gskapka gskapka merged commit 6ed835c into master Feb 26, 2024
5 checks passed
@gskapka gskapka deleted the multi-incremerkles branch February 26, 2024 16:15
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