-
-
Notifications
You must be signed in to change notification settings - Fork 672
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
Support EIP2718 EIP1559 transactions #1604
Comments
Also need this on the MetaMask team to support users who use Trezor. |
this becomes relevant as of the London hardfork, planned for July 2021 |
Is the old transaction format still suported after the hardfork? Edit: Nevermind, the OP says it does. |
How much percent overpay are we talking here for simple transfer and simple ERC20 transfer? |
@prusnak the way that backward compatibility works is that gasPrice is used as the maxFeePerGas and the lack of a maxPriorityFeePerGas means that if the transaction specifies gasPrice of 100 gwei and the baseFee for the block is only 48 gwei then 52 gwei per gas is paid to the miner. Under EIP1559 the fullness of So the answer to your question is: it varies by the network conditions. |
That's really not helpful in deciding whether it's worth implementing this or not. Anyways, the Ethereum community is welcome to contribute a patch via a pull request. We can't afford to spend our resources to development of this feature, but we'll gladly review the change. |
If anyone is interested in adding this to Trezor and funds are the blocker in doing so, please ping me (tim@ethereum.org) and I can help set up a bounty for it. |
I'm gonna take a look at the code to see how difficult it would be to implement this. Will report back with any progress. EDIT: I have started hacking something together here: https://github.com/FrederikBolding/trezor-firmware/tree/eth-eip-1559 EDIT2: Have successfully signed and sent a tx - now need to clean it up 😄 |
After a bit of back and forth over the weekend (and some long nights 😴) it seems that I have a working version of the Trezor firmware that supports EIP 1559. I have opened a pull request (#1653) and we will see what the Trezor reviewers say! |
This is closed - does this mean it works in both Trezor products you actively sell? Or is the Model 1 considered abandoned and won't be updated? |
Waiting for the trezor one. :( |
@scooper4711 @monomesa we are investigating EIP-1559 for Trezor Model One in #1834 |
In the upcoming Ethereum hardfork a new style of transactions is introduced via EIP1559. It leverages the new Typed Transaction Envelope (EIP2718).
It would be great if TREZOR would support this new style of transactions.
Even though legacy transactions will still be supported - TREZOR users would overpay for transaction fees.
The text was updated successfully, but these errors were encountered: