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

eth_call returns insufficient funds in call-simple-transfer test #1907

Closed
1 task done
Tracked by #1579
Rjected opened this issue Mar 22, 2023 · 3 comments · Fixed by #1997
Closed
1 task done
Tracked by #1579

eth_call returns insufficient funds in call-simple-transfer test #1907

Rjected opened this issue Mar 22, 2023 · 3 comments · Fixed by #1997
Assignees
Labels
A-rpc Related to the RPC implementation C-bug An unexpected or incorrect behavior S-needs-investigation This issue requires detective work to figure out what's going wrong

Comments

@Rjected
Copy link
Member

Rjected commented Mar 22, 2023

Describe the bug

The rpc-compat test eth_call/call-simple-transfer returns insufficient funds:

>>  {"jsonrpc":"2.0","id":1,"method":"eth_call","params":[{"from":"0xaa00000000000000000000000000000000000000","gas":"0x186a0","to":"0x0100000000000000000000000000000000000000"},"latest"]}
<<  {"jsonrpc":"2.0","error":{"code":-32003,"message":"Insufficient funds for gas * price + value"},"id":1}
response differs from expected:
 {
-  "error": {
-    "code": -32003,
-    "message": "Insufficient funds for gas * price + value"
-  },
   "id": 1,
   "jsonrpc": "2.0"
+  "result": "0x"
 }

Steps to reproduce

Run hive (#851):

./hive --client reth --sim ethereum/rpc-compat --sim.limit "/eth_call/call-simple-transfer"

Node logs

No response

Platform(s)

No response

What version/commit are you on?

No response

Code of Conduct

  • I agree to follow the Code of Conduct
@Rjected Rjected added C-bug An unexpected or incorrect behavior S-needs-investigation This issue requires detective work to figure out what's going wrong A-rpc Related to the RPC implementation labels Mar 22, 2023
@Rjected Rjected self-assigned this Mar 24, 2023
@Rjected
Copy link
Member Author

Rjected commented Mar 24, 2023

blocked by #1959

see description for the new error

@gakonst
Copy link
Member

gakonst commented Mar 24, 2023

@Rjected I wonder if this is also related to #1951 (comment)?

@Rjected
Copy link
Member Author

Rjected commented Mar 24, 2023

@gakonst it is related, and fails from insufficient funds because of the same code path. After the HistoricalStateProvider fix, the test fails due to sender not an eoa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rpc Related to the RPC implementation C-bug An unexpected or incorrect behavior S-needs-investigation This issue requires detective work to figure out what's going wrong
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants