Chris Hager comments

Results 231 comments of


                                            Chris Hager

Add unit tests to test-cli

test-cli is not actively maintained. are you using it?

Release the RISC-V binary.

I guess we could do a test release build on this branch

Set defaultRelayMinBidEth to 0.05 ETH

50% local blocks seems quite destructive for the PBS and builder market ("would throw quite a bit of chaos into the equation for searchers and order-flow providers").

Bump github.com/ethereum/go-ethereum from 1.13.10 to 1.13.14

@dependabot rebase

[feature] Optimistic v2 submission flow

A new API should be proposed and merged in the relay specs first: https://github.com/flashbots/relay-specs Would it be good if v2 also supports non-optimistic submissions?

Proposal: return `eth_getTransactionCount` from the private mempool when RPC request is signed

Thanks, great writeup. I believe we need to keep the MetaMask workaround in place.

Add SSZ support to builder api

In support of this change. It's nicely backwards-compatible, doesn't add significant risk, decreases latency and could simplify CL flows.

[proposal] require proposer signature for `getHeader` request

Relevant historic thread with more context: https://github.com/flashbots/mev-boost/issues/112 Many relays nowadays support a top-bid websocket stream, which would still keep bids public even if the `getHeader` endpoint requires a signature.

Spurious `no bid for this getPayload payload found`

The error log you mentioned is more of a warning, but doesn't impact the request flow (code is [here](https://github.com/flashbots/mev-boost/blob/c2a5d005161e09d59aa4d42f936d60e2aa65a568/server/service.go#L690)). The reason for the missed slot here was that the bloXroute...

Dynamic Relay Configuration Proposal

This is pretty cool, great work! Will review in detail these days. Btw, love the speed improvements of 25% on the side 🔥 🤠