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

fix(node-loader): Fix anoying error with too large RPC data #3162

Merged
merged 3 commits into from
Aug 24, 2023

Conversation

techraed
Copy link
Member

@techraed techraed commented Aug 23, 2023

Pretty frequently we receive an RPC error, which tell us that RPC payload is too big. This fixes the client side of that error.

Server side is "fixed" with running the node (loaded by gear-node-loader) with --rpc-max-response-size 100 cli param.

@gear-tech/dev

@techraed techraed added A0-pleasereview PR is ready to be reviewed by the team D4-test Autotests, and examples D5-tooling Helper tools and utilities labels Aug 23, 2023
@techraed techraed self-assigned this Aug 23, 2023
Copy link
Member

@ukint-vs ukint-vs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@techraed techraed added A2-mergeoncegreen PR is ready to merge after CI passes and removed A0-pleasereview PR is ready to be reviewed by the team labels Aug 23, 2023
gsdk/src/client.rs Show resolved Hide resolved
@NikVolf NikVolf merged commit e279e6c into master Aug 24, 2023
12 of 15 checks passed
@NikVolf NikVolf deleted the st-fix-large-rpc-data branch August 24, 2023 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A2-mergeoncegreen PR is ready to merge after CI passes D4-test Autotests, and examples D5-tooling Helper tools and utilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants