Skip to content

Ensure SSZ builder works #7000

Closed
Closed
@michaelsproul

Description

@michaelsproul

@pawanjay176 is investigating issues with our SSZ builder implementation, added in:

There may be no fault in our impl, it could be an issue on the mev-boost side.

Either way, we may want to harden our error handling so that we disable SSZ if we get an error response.

We could set ssz_enabled to result.is_ok() here, rather than setting it to true:

self.ssz_enabled.store(true, Ordering::SeqCst);
T::from_ssz_bytes_by_fork(&response_bytes, fork_name)
.map(|data| ForkVersionedResponse {
version: Some(fork_name),
metadata: EmptyMetadata {},
data,
})
.map_err(Error::InvalidSsz)

Similarly, here:

builder
.post_builder_blinded_blocks_ssz(block)
.await
.map_err(Error::Builder)

Metadata

Metadata

Assignees

Labels

builder APIelectraRequired for the Electra/Prague forkv7.0.0New release c. Q1 2025

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions