-
Notifications
You must be signed in to change notification settings - Fork 19
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: VersionedAttestations #447
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit updates the go modules and dependencies, including: - Replacing github.com/probe-lab/hermes with github.com/ethpandaops/hermes - Upgrading github.com/libp2p/go-libp2p to v0.38.1 - Upgrading go.opentelemetry.io/otel and related modules to v1.33.0 - Updating various other dependencies to their latest versions chore: update go.sum dependencies This commit updates the dependencies in go.sum to their latest versions. This includes updates to libraries such as aws-sdk-go-v2, go-openapi, pion, and others. These updates may include bug fixes, performance improvements, and new features. chore: update go.sum dependencies to latest versions This commit updates dependencies in go.sum to their latest versions to ensure compatibility and access to the newest features and security patches. chore: update go.sum dependencies This commit updates the dependencies in go.sum to their latest versions. This ensures that the project is using the most up-to-date and compatible versions of its dependencies. feat: add electra support to beacon block processing This commit adds support for the Electra fork version in beacon block processing. It includes changes to handle Electra blocks in gossipsub beacon block handling, event data extraction, and validator block filtering. This ensures that the system can correctly process and interpret Electra-formatted beacon blocks.
The darwin builds are disabled because cross-compilation is not working.
refactor: use local beacon package instead of github one
The go-eth2-client dependency was updated to the latest version v0.24.1-0.20250219090147-b41ce952806c to address potential bugs and incorporate the newest features.
… handling This commit refactors the attestation event processing logic to improve data handling and error management. The main changes include: - Modified the `NewEventsAttestation` function to accept `spec.VersionedAttestation` instead of `beacon.VersionedAttestation`. - Added error handling to the `NewEventsAttestation` function to return an error if the event is nil or if it fails to get attestation data. - Added `attestationData` field to the `EventsAttestation` struct to store the attestation data. - Modified the `getAdditionalData` function to use the `attestationData` field instead of calling methods on the `event` field. - Modified the `getAdditionalData` function to get the aggregation bits from the event and append the validator index if the attestation is unaggregated. - Updated the `OnAttestation` function in `sentry.go` to use the new `NewEventsAttestation` function and handle errors. These changes improve the data handling and error management of the attestation event processing logic.
This commit introduces support for handling single attestation events in the clmimicry package. It includes the creation of a new file, gossipsub_single_attestation.go, which contains the logic for processing and handling these events. Additionally, the commit updates the go.mod and go.sum files to reflect changes in dependencies. The changes in beacon.go remove the sync distance check.
fix(events_attestation.go): use attestationData instead of event.Phase0.Data to get slot and index
The attestationData field was removed from the EventsAttestation struct. Instead of storing the data directly, the AttestationData method is now used to retrieve it when needed. This change reduces the struct size and avoids storing redundant data.
mattevans
approved these changes
Mar 5, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.