-
Notifications
You must be signed in to change notification settings - Fork 102
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
feat(vald): new EVM config field #1245
Conversation
x/evm/types/config.go
Outdated
Name string `mapstructure:"name"` | ||
RPCAddr string `mapstructure:"rpc_addr"` | ||
WithBridge bool `mapstructure:"start-with-bridge"` | ||
RPCModuleSupport bool `mapstructure:"rpc-module-support"` |
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.
This name is quite weird. It's basically saying if it's set to true
, then do not try to detect whether or not it's beyond the standard ethereum jsonrpc. Maybe something like enable-rpc-detection
.
Co-authored-by: Sammy Liu <sammy.liu@axelar.network>
Co-authored-by: Sammy Liu <sammy.liu@axelar.network>
Name string `mapstructure:"name"` | ||
RPCAddr string `mapstructure:"rpc_addr"` | ||
WithBridge bool `mapstructure:"start-with-bridge"` | ||
EnableRPCDetection bool `mapstructure:"enable-rpc-detection"` |
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.
Just wanna confirm before you merge, does this mean that EnableRPCDetection
is optional in the config.toml
file and if one doesn't have it there it would default to true
?
Description
Upon testing non-standard rpc methods (e.g., Moonbeam's
chain_getFinalizedHead
), ganache returns error code -32000 (server error) instead of -32601 (method not found error). In order to be able to continue using ganache for local testing (as well as to account for other possible situations where we wouldn't want to support non-standard rpc functionality), this PR adds a new field in evm config params calledrpc-module-support
, which allows vald to skip detecting non-standard rpc functionality.Todos
Steps to Test
Expected Behaviour
Other Notes