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

[3.2] Add --http-max-in-flight-requests to http_plugin option #561

Merged
merged 4 commits into from
Jun 30, 2022

Conversation

linh2931
Copy link
Member

Backport EOSIO/eos#9431, resolve #431.

Notes about backport: current http_plugin.cpp was already ahead of EOSIO/eos#9431 and the PR had several commits going back and forth changes. Those made cherry-picking all the commits impossible. The first main commit was cherry-picked. The remaining ones were manually pulled in.

From original PR:
Added a new option for 'http-plugin': http-max-in-flight-requests. It restricts the maximum requests the flight can hold if --http-max-in-flight-requests = positive number. If the limit exceeds, it sends a 429 error to the client. if --http-max-in-flight-requests= -1, there is no limit so the feature is turned off. It protects non-producing nodeos against the flood of requests when it cannot get new blocks.

@linh2931 linh2931 requested review from heifner and spoonincode June 28, 2022 23:09
@linh2931 linh2931 self-assigned this Jun 28, 2022
@linh2931 linh2931 merged commit 60a5ee3 into main Jun 30, 2022
@linh2931 linh2931 deleted the http_max_inflight_reqs branch June 30, 2022 16:55
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.

Backport: Add --http-max-in-flight-requests to http_plugin option
3 participants