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

Lacking BASEFEE instruction support on neon-evm #51

Open
pcy190 opened this issue Nov 14, 2022 · 1 comment
Open

Lacking BASEFEE instruction support on neon-evm #51

pcy190 opened this issue Nov 14, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@pcy190
Copy link

pcy190 commented Nov 14, 2022

Bug Description

Since neon-evm uses the old fork of sputnik-vm [1], it doesn't support the proposal after london fork.

For example, when encounter BASEFEE instruction, neon-evm treats it as an invalid instruction and revert the execution. This may lead to the misbehavior of the contract which targets at the newer fork (e.g., London fork ).

[1]https://github.com/neonlabsorg/evm/tree/3ec2db790354baf39f464563751079894eeece26

Actual Behavior

neon-evm reverts the execution when encounters BASEFEE instruction.

Expected Behavior

neon-evm properly handles the BASEFEE instruction under EIP-1559.

@pcy190 pcy190 added the bug Something isn't working label Nov 14, 2022
@gigimon
Copy link
Collaborator

gigimon commented Sep 18, 2023

Hello, sorry for a long reply. We are waiting right now to support eip-1559.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants