engine payload bodies rpc endpoints#6644
Conversation
hexoscott
force-pushed
the
get-payload-bodies
branch
2 times, most recently
from
00d05f0 to
624cd5c
Compare
hexoscott
force-pushed
the
get-payload-bodies
branch
2 times, most recently
from
ef541bd to
25a7201
Compare
cmd/rpcdaemon/commands/engine_api.go
Outdated
There was a problem hiding this comment.
Slot numbers in CL are different from block numbers in the EL. This field refers to the block rather than the slot number (see Item 6 of the spec). Please rename to StartBlockNumber or simply Start. And it's "start" in json per the spec.
There was a problem hiding this comment.
This is where I got the name for the json from, it mentions not to confuse this with the block number which we're OK on. I think we need to keep the start_slot, but can rename the field name no problem?
There was a problem hiding this comment.
Turns out we don't need this type at all anyway, the last comment here negates the need for it
ethdb/privateapi/ethbackend.go
Outdated
There was a problem hiding this comment.
Trailing null blocks (blocks after the latest available) should be truncated (see Item 3 of the spec). In our database we don't allow any gaps in canonical blocks, so it's OK to truncate starting from the first missing block.
There was a problem hiding this comment.
OK good to know. There were some comments on this when I initially looked at the PR, looks like that has been finalised now so will update our code.
cmd/rpcdaemon/commands/engine_api.go
Outdated
There was a problem hiding this comment.
This method should have 2 parameters (start, count) per the spec, not one, similar to how ForkchoiceUpdatedV1 has 2 parameters.
hexoscott
force-pushed
the
get-payload-bodies
branch
from
25a7201 to
e00409d
Compare
2 participants
Very basic implementation for get payload bodies rpc calls. Once we have Hive tests for these calls I can pick this back up and work through any issues.
Implementation of ethereum/execution-apis#352.