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

[FEATURE] Extend rate limiter concept to beyond just auth failure #4652

Open
derek-ho opened this issue Aug 16, 2024 · 2 comments
Open

[FEATURE] Extend rate limiter concept to beyond just auth failure #4652

derek-ho opened this issue Aug 16, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@derek-ho
Copy link
Contributor

Is your feature request related to a problem?
The concept of rate limiting shouldn't be limited only to login failures, it should also be able to be configured for any type of requests (success or failure).
What solution would you like?
A way to configure rate limiting for successful requests as well (failure existing today).
What alternatives have you considered?
None
Do you have any additional context?
No

@derek-ho derek-ho added enhancement New feature or request untriaged Require the attention of the repository maintainers and may need to be prioritized labels Aug 16, 2024
@stephen-crawford
Copy link
Collaborator

[Triage] Hi @derek-ho, thanks for filing this issue. @reta to follow up with some more comments on this topic. Going to leave without the triaged label for the time being.

@stephen-crawford stephen-crawford removed the untriaged Require the attention of the repository maintainers and may need to be prioritized label Aug 19, 2024
@reta
Copy link
Collaborator

reta commented Aug 19, 2024

I think we have to decide if this is an infrastructure concern (API gateway, mesh, ...) or is is necessary feature of the OpenSearch cluster? I believe rate limiting is somewhat solved problem in 99% of the deployments out there and not something OpenSearch has to (re)implement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants