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

fix: allow a previously underpriced transaction back in after a timeout #23

Conversation

andyzhang2023
Copy link
Contributor

This issue had been reported to official repo in go-ethereum, here is it #28018.
And the fix solution had been adopted by official team, with the merged pr #28097.

Copy link
Contributor

@bendanzhentan bendanzhentan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to introduce these updates via cherry-pick? It keeps the source author when use cherry-pick.

@andyzhang2023
Copy link
Contributor Author

Is it possible to introduce these updates via cherry-pick? It keeps the source author when use cherry-pick.

Yes, i redo it with cherry-pick here: #34,

sunny2022da added a commit to sunny2022da/op-geth that referenced this pull request Aug 13, 2024
This PR refine and clean up the code of PEVM

Co-authored-by: Sunny <[email protected]>
sunny2022da added a commit to sunny2022da/op-geth that referenced this pull request Sep 25, 2024
This PR refine and clean up the code of PEVM

Co-authored-by: Sunny <[email protected]>
sunny2022da added a commit to sunny2022da/op-geth that referenced this pull request Oct 11, 2024
This PR refine and clean up the code of PEVM

Co-authored-by: Sunny <[email protected]>
welkin22 pushed a commit that referenced this pull request Oct 22, 2024
This PR refine and clean up the code of PEVM

Co-authored-by: Sunny <[email protected]>
sunny2022da added a commit to sunny2022da/op-geth that referenced this pull request Nov 14, 2024
This PR refine and clean up the code of PEVM

Co-authored-by: Sunny <[email protected]>
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