Closed yangby-cryptape closed 9 months ago
The filed chainId is required for latest transactions:
chainId
TransactionLegacyUnsigned
Transaction2930Unsigned
Transaction1559Unsigned
For JSON-RPC method eth_estimateGas:
eth_estimateGas
Ethereum will accept this field. (ref: Ethereum JSON-RPC Specification)
But Axon won't.
When a request contains chainId, Axon will throw an error.
https://github.com/axonweb3/axon/blob/f692b865bda85cbd0c8f028f8aca8ae0d793465b/core/api/src/jsonrpc/web3_types.rs#L303-L322
Other fields in JSON-RPC methods required to be checked, too.
In fact, hardhat (ethers.js) always sends this field when call eth_estimateGas.
Why does it found until now? Because, all tests call this method manually.
For examples: THIS.
Description
The filed
chainId
is required for latest transactions:TransactionLegacyUnsigned
,chainId
is optional.Transaction2930Unsigned
,chainId
is required.Transaction1559Unsigned
,chainId
is required.For JSON-RPC method
eth_estimateGas
:Ethereum will accept this field. (ref: Ethereum JSON-RPC Specification)
But Axon won't.
When a request contains
chainId
, Axon will throw an error.https://github.com/axonweb3/axon/blob/f692b865bda85cbd0c8f028f8aca8ae0d793465b/core/api/src/jsonrpc/web3_types.rs#L303-L322
In Further
Other fields in JSON-RPC methods required to be checked, too.
In fact, hardhat (ethers.js) always sends this field when call
eth_estimateGas
.Why does it found until now? Because, all tests call this method manually.
For examples: THIS.