-
Notifications
You must be signed in to change notification settings - Fork 35
Flashblocks #98
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
base: main
Are you sure you want to change the base?
Flashblocks #98
Conversation
* deps: alloy 0.12.5 * fix reth dep * lint
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
Wen merge? :) |
/// The transactions of the block. | ||
pub transactions: Vec<Bytes>, | ||
/// Array of [`Withdrawal`] enabled with V2 | ||
pub withdrawals: Vec<Withdrawal>, |
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.
the withdrawals should be immutable too no?
- Added gas_delta and tx_count_delta to payload metrics in get_payload_v3 and get_payload_v4. - Updated MetricsSpanProcessor to include INVALID_PAYLOAD tracking. - Refactored error handling in RpcClient to log invalid payloads instead of incrementing counters directly.
metrics updates
* bump alloys version * bump dev deps
} | ||
|
||
#[derive(Parser, Debug)] | ||
pub struct FlashblocksArgs { |
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.
could you add docs to README and .env.example
let receiver = FlashblocksReceiverService::new(builder_url, tx)?; | ||
|
||
tokio::spawn(async move { | ||
let _ = receiver.run().await; |
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.
is there a need for error handling here?
No description provided.