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

Added an option to avoid interrupting the request pipeline #425

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dastankgz
Copy link

Hello!
We use AspNetCoreRateLimit with Redis (as a distributed counter store). And in this use case, Redis might not be available. This causes the target service to stop servicing the requests (due to exception in RateLimitMiddleware). But it is critical for us to keep serving requests even if there are some rate limiting issues (Redis not available or some other issue).
It would be ideal if there was an option to disable interrupting the request pipeline due to rate limiting issues.
Please consider this PR.

@dastankgz
Copy link
Author

Hi @stefanprodan, please take a look at the PR, it would be great to have this in lib so we don't have to keep a separate fork.

@vmandic
Copy link

vmandic commented May 26, 2023

This would be excellent... @cristipufu any input please? Seems PRs are on hold for a while now? Is this still maintained? Thank you so much for all the effort! The new .NET7 lib is not really feature rich as this one and we still remain using this one.

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.

2 participants