-
Notifications
You must be signed in to change notification settings - Fork 1
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
working devnet #8
Conversation
fix: get header with proof resp type
fix: verify signature with domain
fix: deserialize signed builder bid with proofs
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.
Great!
let mut builder_infos = db.get_all_builder_infos().await.expect("failed to load builder infos"); | ||
|
||
builder_infos.push(BuilderInfoDocument { | ||
pub_key: PublicKey::try_from( | ||
Bytes::from_hex("0xaa1488eae4b06a1fff840a2b6db167afc520758dc2c8af0dfb57037954df3431b747e2f900fe8805f05d635e9a29717b").unwrap().as_ref() | ||
).expect("failed to convert to public key"), | ||
builder_info: BuilderInfo { | ||
collateral: U256::MAX / U256::from(4), | ||
is_optimistic: true, | ||
builder_id: Some("Bolt".to_string()) | ||
} | ||
}); |
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.
This is currently hardcoded to avoid inserting builders manually each time we start the devnet via an SQL query. A configuration-based approach is explored in this PR: gattaca-com#41. For now we can keep it as it this and change it later.
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.
Some nits!
This PR contains Kurtosis devnet integration errors.