-
Notifications
You must be signed in to change notification settings - Fork 76
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
adds client/server impl for relay-specs
APIs
#109
Conversation
pub trait BlindedBlockRelayer { | ||
async fn get_proposal_schedule(&self) -> Result<Vec<ProposerSchedule>, Error>; | ||
|
||
// TODO: support cancellations? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: from offline discussions, cancellations appear necessary to attract builders
#[cfg_attr(feature = "serde", derive(serde::Serialize, serde::Deserialize))] | ||
pub struct SignedBidSubmission { | ||
pub message: BidTrace, | ||
pub execution_payload: ExecutionPayload, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see execution_payload
in the relay-specs
, but I guess it is part of the flashbots relay response?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
its missing from the example and needs to be updated
its literally the entire point to accept a payload here
/// Spawns the server on a new task returning the handle for it | ||
pub fn spawn(&self) -> JoinHandle<()> { | ||
let server = self.serve(); | ||
let address = server.local_addr(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: prefer addr
to be consistent with local_addr
and variable in serve
above
alternatively, just write tracing::info!("listening at {}...", server.local_addr());
below
No description provided.