-
Notifications
You must be signed in to change notification settings - Fork 1
fix: update to eip 7594 blob handling #202
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?
Conversation
|
isn't this already ticketed for @Evalir ? where are we at there? |
|
@prestwich this ticket hasn't really been spec'ed yet. I've asked a question or two there which we haven't answered. while the current PR works I'd recommend we hash it out on linear before acting further, since we shipped some peripheral alloy work to make this easier |
|
Follow up work spec'd in ENG-1649 |
|
This PR at latest commit has been tested and confirmed working in devent with the roundtrip script. |
| use tracing::{Instrument, debug, debug_span, error, instrument}; | ||
|
|
||
| /// Type alias for a EIP 7594 compatible blob sidecar transaction envelope. | ||
| type SubmitEnvelope7594 = EthereumTxEnvelope<TxEip4844Variant<BlobTransactionSidecarEip7594>>; |
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.
Maybe just TxEnvelope7594?
| let sidecar = self.build_sidecar().await?; | ||
| let input = self.build_input().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.
join!(self.build_sidecar(), self.build_input() to chain both

fix: update to eip 7594 blob handling
This PR changes the
prep_transactionto create a EIP-7594 compatible blob sidecar.Closes ENG-1582
Related to ENG-1562