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
Cancun updates #253
Cancun updates #253
Conversation
9c37511
to
8f86c60
Compare
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.
lgtm! Nice reorg
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.
Looks good to me. Just left a few nits
bacefd3
to
438c641
Compare
New API for generating commitments and proofs from blobs added with pydantic models for validation. RLP serialization and deserialization with the edit: more tests added |
- Re-organize typed transactions as a directory with each `_TypedTransactionImplementation` as its own file. - Add support for `BlobTransaction` (type = 3). - Add relevant tests. Also: - Normalize transaction dict for ``TypedTransaction.as_dict()`` so that lists or tuples are converted to tuples for consistency and ease of comparison.
- When a blob transaction is signed, it needs to contain the blob information along with commitments and proofs for the consensus layer. We need to be able to serialize and deserialize this information. - Use different serializers for execution layer style `TransactionPayloadBody` blob transactions and for `PooledTransaction` blob transactions.
- Add some testing around blob transaction `blobVersionedHashes` validation when present in tx dict compared to the computed values from blobs. - Add testing around the computing of commitment, proofs, and versioned hashes. - Add testing around the blob count limits for a blob carrying transaction.
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.
Apart from the comment on fields to add to _TypedTransactionImplementation
, lgtm!
What was wrong?
3
blob transactionsaccessList
orblobVersionedHashes
within a dict all normalize to tuples for consistency and ease of comparison.Todo:
PooledTransaction
APICute Animal Picture