-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: Update ppom_release to fix fail ppom security validation check #8605
Conversation
Updating ppom fixes an ongoing issue caused by an updated of geth in the infura nodes
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
E2E test started on Bitrise: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/9b928279-78df-427f-8a8c-af7fd8dcfda2 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8605 +/- ##
=======================================
Coverage 41.06% 41.06%
=======================================
Files 1247 1247
Lines 30351 30351
Branches 2963 2963
=======================================
Hits 12464 12464
Misses 17142 17142
Partials 745 745 ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
Missing release label release-7.16.0 on PR. Adding release label release-7.16.0 on PR and removing other release labels(release-7.17.0), as PR was cherry-picked in branch 7.16.0. |
cherry-pick PR into release: #8609 |
Description
A recent geth update and how a eth_createAccessList request is handled when passed certain variables, created an incompatibility with ppom causing it to fail with
gasPrice must be non-zero after london fork, data: None)
.The incompatibility of ppom and the latest changes in geth is handled in v1.4.1 of ppom_release.
Related issues
Fixes: #8589
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist