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

go-ethereum>=1.13.0 is not compatible because eth_call API has changed (data became input for call arguments) #1825

Closed
Darkness4 opened this issue Feb 16, 2024 · 3 comments · Fixed by #1852
Assignees
Labels
bug Something isn't working epic:eth-compatibility
Milestone

Comments

@Darkness4
Copy link

Darkness4 commented Feb 16, 2024

Describe the bug

Hi, it seems that go-ethereum has decided to use the input field instead of the data field on their JSON-RPC client. It would be great if SKALE could support the input field and still be backwards compatible. Apparently, the specs changed 6 years ago. I don't know what other changes have been added.

Relevent issues: ethereum/go-ethereum#28608, ethereum/go-ethereum#28078 and ethereum/go-ethereum#15628

Revelent docs:

To Reproduce
Steps to reproduce the behavior:

  1. eth_estimateGas, eth_call... with go-ethereum 1.13.0.
  2. Gas estimation throws EVM revert instruction without description message

Expected behavior

  1. Launch a transaction/call.
@Darkness4 Darkness4 changed the title go-ethereum>1.13.0 is not compatible because eth API has changed (data became input for call arguments) go-ethereum>=1.13.0 is not compatible because eth API has changed (data became input for call arguments) Feb 16, 2024
@Darkness4 Darkness4 changed the title go-ethereum>=1.13.0 is not compatible because eth API has changed (data became input for call arguments) go-ethereum>=1.13.0 is not compatible because eth_call API has changed (data became input for call arguments) Feb 16, 2024
@DmytroNazarenko DmytroNazarenko added this to the SKALE 2.4 milestone Feb 22, 2024
@DmytroNazarenko DmytroNazarenko added the bug Something isn't working label Feb 22, 2024
@DmytroNazarenko
Copy link
Collaborator

Hi @Darkness4 ! Thanks for reporting this issue, it is addressed to the following release

@PolinaKiporenko PolinaKiporenko moved this to Ready For Pickup in SKALE Engineering 🚀 Feb 23, 2024
@DmytroNazarenko DmytroNazarenko moved this from Ready For Pickup to In Progress in SKALE Engineering 🚀 Mar 18, 2024
DmytroNazarenko added a commit that referenced this issue Mar 18, 2024
DmytroNazarenko added a commit that referenced this issue Mar 18, 2024
DmytroNazarenko added a commit that referenced this issue Mar 18, 2024
DmytroNazarenko added a commit that referenced this issue Mar 19, 2024
@DmytroNazarenko DmytroNazarenko linked a pull request Mar 19, 2024 that will close this issue
@DmytroNazarenko DmytroNazarenko moved this from In Progress to Code Review in SKALE Engineering 🚀 Mar 19, 2024
DmytroNazarenko added a commit that referenced this issue Mar 20, 2024
@github-project-automation github-project-automation bot moved this from Code Review to Ready For Release Candidate in SKALE Engineering 🚀 Mar 21, 2024
@PolinaKiporenko
Copy link
Contributor

PolinaKiporenko commented May 20, 2024

skaled:3.19.0-beta.0

@PolinaKiporenko PolinaKiporenko moved this from Ready For Release Candidate to Merged To Release Candidate in SKALE Engineering 🚀 May 20, 2024
@EvgeniyZZ EvgeniyZZ moved this from Merged To Release Candidate to QA in SKALE Engineering 🚀 May 20, 2024
@oleksandrSydorenkoJ
Copy link

Network Regression
Skale_schain carefree-knowing-sarin
skaled:3.19.0-beta.1

Example contract

// SPDX-License-Identifier: GPL-3.0

pragma solidity >=0.8.20;

contract Push0Test {
    function f() public pure returns (uint256){
        return 1;
    }
}

Example request "input" and "data"

 curl http://localhost:10067   -X POST   -H "Content-Type: application/json"   --data '{"id":1610870259,"jsonrpc":"2.0","method":"eth_call","params":[{"input":"0x26121ff0","from":"0x71cBE3fedE33905d4D1Bf2Bd51f9d4A62375e659","to":"0xA4cE8db7791E107a46615231F8DAe594d552aA35","value":"0x0"},"latest"]}'  
 
{"id":1610870259,"jsonrpc":"2.0","result":"0x0000000000000000000000000000000000000000000000000000000000000001"} 

curl http://localhost:10067   -X POST   -H "Content-Type: application/json"   --data '{"id":1610870259,"jsonrpc":"2.0","method":"eth_call","params":[{"data":"0x26121ff0","from":"0x71cBE3fedE33905d4D1Bf2Bd51f9d4A62375e659","to":"0xA4cE8db7791E107a46615231F8DAe594d552aA35","value":"0x0"},"latest"]}'  

{"id":1610870259,"jsonrpc":"2.0","result":"0x0000000000000000000000000000000000000000000000000000000000000001"}

estimate_gas

curl http://localhost:10067   -X POST   -H "Content-Type: application/json"   --data '{"id":7,"jsonrpc":"2.0","method":"eth_estimateGas","params":[{"from":"0x71cBE3fedE33905d4D1Bf2Bd51f9d4A62375e659","gasPrice":"0x186a0","input":"0x","nonce":"0xa4c","to":"0x71cBE3fedE33905d4D1Bf2Bd51f9d4A62375e659","value":"0x0"}]}'  

{"id":7,"jsonrpc":"2.0","result":"0x5208"}

 curl http://localhost:10067   -X POST   -H "Content-Type: application/json"   --data '{"id":7,"jsonrpc":"2.0","method":"eth_estimateGas","params":[{"from":"0x71cBE3fedE33905d4D1Bf2Bd51f9d4A62375e659","gasPrice":"0x186a0","data":"0x","nonce":"0xa4c","to":"0x71cBE3fedE33905d4D1Bf2Bd51f9d4A62375e659","value":"0x0"}]}'  

{"id":7,"jsonrpc":"2.0","result":"0x5208"}

@EvgeniyZZ EvgeniyZZ moved this from QA to Done in SKALE Engineering 🚀 Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working epic:eth-compatibility
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants