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.

Sign up for GitHub

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

Jump to bottom

feat(gateway): implement request concurrency limit #887

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat(gateway): implement request concurrency limit #887

wants to merge 2 commits into from

Conversation

gitsrc
Copy link
Contributor

@gitsrc gitsrc commented Mar 19, 2025

This commit introduces a mechanism to limit the number of concurrent in-flight HTTP requests to the gateway.

  • Implements a configurable limit for concurrent requests.
  • Returns HTTP 503 Service Unavailable with Retry-After header when the limit is reached.
  • Includes a default limit to protect against traffic spikes when exposed without a reverse proxy.

This change enhances gateway's resilience and prevents overload by limiting concurrent requests.

#881

@gitsrc gitsrc requested review from lidel and a team as code owners March 19, 2025 11:30
@hsanjuan hsanjuan added the need/maintainers-input Needs input from the current maintainer(s) label Mar 25, 2025
@hsanjuan
Copy link
Contributor

Triage notes: this will be triaged as part Kubo v0.35 (@lidel )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lidel lidel Awaiting requested review from lidel lidel is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
need/maintainers-input Needs input from the current maintainer(s)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gitsrc @hsanjuan