Skip to content
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

Adopt alloy types for transaction types #802

Open
Wodann opened this issue Feb 11, 2025 · 0 comments
Open

Adopt alloy types for transaction types #802

Wodann opened this issue Feb 11, 2025 · 0 comments

Comments

@Wodann
Copy link
Member

Wodann commented Feb 11, 2025

While working on #786, I investigated using alloy-rs's TxEip7702 type, but wasn't able to use those types side-by-side with our existing transaction types.

TxEip7702 would be equivalent to our transaction::request::Eip7702. To obtain a signed transaction, alloy uses Signed<TxEip7702>, which adds pre-computation of a tranaction_hash, RLP encoding/decoding, etc.

One aspect of EDR that wasn't easily transferred to types from alloy was the ability to have a "fakeable signature" which we support.

Definition of Done
Investigate and refactor transaction types to reuse alloy types and tests

@github-project-automation github-project-automation bot moved this to Inbox in EDR Feb 11, 2025
@Wodann Wodann moved this from Inbox to Todo in EDR Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

1 participant