Skip to content
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

blocksv3: add consensus value #358

Merged
merged 2 commits into from
Oct 1, 2023

Commits on Sep 20, 2023

  1. blocksv3: add consensus value

    In order to compare blocks coming from two sources, a validator client
    must consider both consensus and execution profit - without consensus
    reward information, the VC must fall back on unreliable heuristics that
    can be biased towards inefficient consensus packing, specially when
    execution payloads are the same which they are likely to be if the block
    producer is using the same mev relay for both beacon nodes.
    
    The consensus computation is currently not a necessary part of block
    construction but given that blocks are constructed over a state, the
    information necessary to compute the is already present at block
    construction time.
    
    This PR suggests a mandatory value to make validator client multiplexing
    easier at the expense of beacon node complexity - if instead it is
    optional, we are back to where we started where blocks cannot be
    compared across implementations.
    
    The value has to be trusted, ie the BN could be report an inaccurate
    value but this is not a new risk.
    arnetheduck committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    c6488cf View commit details
    Browse the repository at this point in the history

Commits on Sep 28, 2023

  1. payload->block

    arnetheduck committed Sep 28, 2023
    Configuration menu
    Copy the full SHA
    bcaa08c View commit details
    Browse the repository at this point in the history