Skip to content

Ensure SSZ builder works #7000

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)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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