• Marius van der Wijden's avatar
    internal/ethapi: return revert reason for eth_call (#21083) · 0b3f3be2
    Marius van der Wijden authored
    * internal/ethapi: return revert reason for eth_call
    
    * internal/ethapi: moved revert reason logic to doCall
    
    * accounts/abi/bind/backends: added revert reason logic to simulated backend
    
    * internal/ethapi: fixed linting error
    
    * internal/ethapi: check if require reason can be unpacked
    
    * internal/ethapi: better error logic
    
    * internal/ethapi: simplify logic
    
    * internal/ethapi: return vmError()
    
    * internal/ethapi: move handling of revert out of docall
    
    * graphql: removed revert logic until spec change
    
    * rpc: internal/ethapi: added custom error types
    
    * graphql: use returndata instead of return
    
    Return() checks if there is an error. If an error is found, we return nil.
    For most use cases it can be beneficial to return the output even if there
    was an error. This code should be changed anyway once the spec supports
    error reasons in graphql responses
    
    * accounts/abi/bind/backends: added tests for revert reason
    
    * internal/ethapi: add errorCode to revert error
    
    * internal/ethapi: add errorCode of 3 to revertError
    
    * internal/ethapi: unified estimateGasErrors, simplified logic
    
    * internal/ethapi: unified handling of errors in DoEstimateGas
    
    * rpc: print error data field
    
    * accounts/abi/bind/backends: unify simulatedBackend and RPC
    
    * internal/ethapi: added binary data to revertError data
    
    * internal/ethapi: refactored unpacking logic into newRevertError
    
    * accounts/abi/bind/backends: fix EstimateGas
    
    * accounts, console, internal, rpc: minor error interface cleanups
    
    * Revert "accounts, console, internal, rpc: minor error interface cleanups"
    
    This reverts commit 2d3ef53c5304e429a04983210a417c1f4e0dafb7.
    
    * re-apply the good parts of 2d3ef53c53
    
    * rpc: add test for returning server error data from client
    Co-authored-by: 's avatarrjl493456442 <garyrong0905@gmail.com>
    Co-authored-by: 's avatarPéter Szilágyi <peterke@gmail.com>
    Co-authored-by: 's avatarFelix Lange <fjl@twurst.com>
    0b3f3be2
Name
Last commit
Last update
.github Loading commit data...
accounts Loading commit data...
build Loading commit data...
cmd Loading commit data...
common Loading commit data...
consensus Loading commit data...
console Loading commit data...
contracts/checkpointoracle Loading commit data...
core Loading commit data...
crypto Loading commit data...
docs/audits Loading commit data...
eth Loading commit data...
ethclient Loading commit data...
ethdb Loading commit data...
ethstats Loading commit data...
event Loading commit data...
graphql Loading commit data...
internal Loading commit data...
les Loading commit data...
light Loading commit data...
log Loading commit data...
metrics Loading commit data...
miner Loading commit data...
mobile Loading commit data...
node Loading commit data...
p2p Loading commit data...
params Loading commit data...
rlp Loading commit data...
rpc Loading commit data...
signer Loading commit data...
swarm Loading commit data...
tests Loading commit data...
trie Loading commit data...
whisper Loading commit data...
.dockerignore Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gitmodules Loading commit data...
.golangci.yml Loading commit data...
.mailmap Loading commit data...
.travis.yml Loading commit data...
AUTHORS Loading commit data...
COPYING Loading commit data...
COPYING.LESSER Loading commit data...
Dockerfile Loading commit data...
Dockerfile.alltools Loading commit data...
Makefile Loading commit data...
README.md Loading commit data...
SECURITY.md Loading commit data...
appveyor.yml Loading commit data...
circle.yml Loading commit data...
fuzzbuzz.yaml Loading commit data...
go.mod Loading commit data...
go.sum Loading commit data...
interfaces.go Loading commit data...