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

[rpc] disable trace api in rpc #3928

Merged
merged 1 commit into from
Nov 12, 2021

Conversation

JackyWYX
Copy link
Contributor

Issue

Trace APIs are heavy RPCs. Currently, the backend nodes are receiving a large amount of these RPC method calls, causing bad RPC service performance.

We will disable the Trace methods for now, meanwhile it will effect the explorer contract code verification. We will enable the methods in private methods later.

Test

Tested locally.

curl --location --request POST 'localhost:9500' \
--header 'Content-Type: application/json' \
--data-raw '{
    "jsonrpc": "2.0",
    "id": 1,
    "method": "hmyv2_traceTransaction",
    "params": []
}'
{"jsonrpc":"2.0","id":1,"error":{"code":-32601,"message":"the method hmyv2_traceTransaction does not exist/is not available"}}

@LeoHChen Please verify the result before deployment. Thanks.

@JackyWYX JackyWYX requested review from rlan35 and LeoHChen November 12, 2021 02:00
@LeoHChen LeoHChen merged commit 4154b53 into harmony-one:main Nov 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants