Skip to content

Add RateLimit middleware docs. #8063

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
Open
wants to merge 1 commit into
base: 5.next
Choose a base branch
from
Open

Add RateLimit middleware docs. #8063

wants to merge 1 commit into from

Conversation

@dereuromark
Copy link
Member

@dereuromark dereuromark commented Aug 15, 2025 •
edited
Loading

@dereuromark dereuromark added this to the 5.next milestone Aug 15, 2025
@dereuromark dereuromark mentioned this pull request Aug 15, 2025
Merged
@othercorey
Copy link
Member

othercorey commented Nov 9, 2025

Need to fix conflicts then merge.

@ADmad
Copy link
Member

ADmad commented Nov 10, 2025 •
edited
Loading

There's no mention of the RateLimiterInterface::reset() in the docs. Also this method isn't used by either the RateLimitMidddleware not the strategy classes themselves. So how/when should one use it?

@ADmad ADmad mentioned this pull request Nov 10, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

5.next

Development

Successfully merging this pull request may close these issues.

4 participants

@dereuromark @othercorey @ADmad