-
Notifications
You must be signed in to change notification settings - Fork 298
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 missing electra attestation processing committee check #8901
add missing electra attestation processing committee check #8901
Conversation
humm looks like we do things slightly differently there.. for instance where is |
That's done:
As discussed, I'll push some other changes to make it more aligned with the spec. |
...src/main/java/tech/pegasys/teku/spec/logic/versions/electra/block/BlockProcessorElectra.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Just a var name suggestion
PR Description
Add missing committee attester check when processing Electra attestations:
https://github.com/ethereum/consensus-specs/blob/31cd9cbec2d93e5a6df84dcee9ca9e8aa77803ee/specs/electra/beacon-chain.md?plain=1#L1330
Fixed Issue(s)
fixes #8892
Documentation
doc-change-required
label to this PR if updates are required.Changelog