-
Notifications
You must be signed in to change notification settings - Fork 48
Problem: no max gas cap for grpc eth_call #586
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
base: release/v1.4.x-2
Are you sure you want to change the base?
Problem: no max gas cap for grpc eth_call #586
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release/v1.4.x-2 #586 +/- ##
====================================================
- Coverage 46.15% 46.14% -0.01%
====================================================
Files 181 181
Lines 18182 18180 -2
====================================================
- Hits 8391 8389 -2
Misses 9071 9071
Partials 720 720
🚀 New features to boost your workflow:
|
Are we sure this is non-breaking if we change it at |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, seems only used in query related simulation operations, will also affect eth_estimateGas
debug_traceCall
19380d9
to
50df3af
Compare
50df3af
to
bf350e8
Compare
Closes: #XXX
Description
For contributor use:
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerFor admin use:
WIP
,R4R
,docs
, etc)