-
Notifications
You must be signed in to change notification settings - Fork 20.2k
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
eth/catalyst,beacon/engine: implement GetClientVersionV1 #28915
Conversation
Spec has been changed to return an array of versions. |
Okay the PR was merged ethereum/execution-apis#517, this should be good to go! |
Code: engine.ClientCode, | ||
Name: engine.ClientName, | ||
Version: params.VersionWithMeta, | ||
Commit: hexutil.Encode(commit), |
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.
This will add 0x
in the commit. Is this seriously what they decided on within the spec?
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.
It's type DATA
which should have the 0x
prefix https://github.com/ethereum/execution-apis/blob/main/src/engine/common.md#encoding
Looks like you forgot to add it into the list of capabilites |
This implements the new method
engine_getClientVersionV1
. Still waiting it to be merged here: ethereum/execution-apis#517