Skip to content

fix(t8n): implement signature recovery for transactions#151

Open
huahuayu wants to merge 1 commit intomegaeth-labs:mainfrom
huahuayu:fix/t8n-signature-recovery
Open

fix(t8n): implement signature recovery for transactions#151
huahuayu wants to merge 1 commit intomegaeth-labs:mainfrom
huahuayu:fix/t8n-signature-recovery

Conversation

@huahuayu
Copy link

This PR implements signature recovery for transactions in the t8n tool. Previously, it only supported transactions with a secret_key provided in the input. Now, it correctly recovers the sender address from the v, r, s signature components using MegaTxEnvelope::recover_signer.

Changes:

  • Added MegaTxEnvelope to imports in cmd.rs.
  • Implemented recovery logic for Legacy, EIP-2930, EIP-1559, and EIP-7702 transactions.

@Troublor
Copy link
Collaborator

Thanks, t8n was meant to be used to run ethereum-tests, but it has been broken for months. I will revisit this next month

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments