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

Unblock IP from rate limiter? #2856

Open
sephentos opened this issue Jun 13, 2023 · 3 comments · May be fixed by #2932
Open

Unblock IP from rate limiter? #2856

sephentos opened this issue Jun 13, 2023 · 3 comments · May be fixed by #2932
Labels
type/enhancement Enhances existing functionality

Comments

@sephentos
Copy link

[2023-06-13 02:13:41,739] WARNING in limiter: Authentication attempt from xxx.xxx.xxx.xxx has been rate-limited.

Is there a way to unblock the IP address in the rate limiter?
I've searched the documentation and issues but couldn't find anything.

@ghostwheel42
Copy link
Contributor

Yes, it is possible and should probably be documented or added to the cli interface.
You can do it using redis-cli:

# list the limited networks (this is not the IP, but only the network part according to AUTH_RATELIMIT_IP_V4_MASK
docker compose exec redis redis-cli -n 2 --scan --pattern 'LIMITER/auth-ip/*'
# delete an entry
IP=8.8.8.8; docker compose exec redis redis-cli -n 2 --scan --pattern "LIMITER/auth-ip/${IP}/*" | xargs -r docker compose exec -T redis redis-cli -n 2 DEL

@ghostwheel42 ghostwheel42 added the type/enhancement Enhances existing functionality label Jun 23, 2023
@DjVinnii
Copy link
Contributor

I also ran into this problem recently. The Redis CLI was useful, however it would be a nice addition to show and manage the rate limited IP's from the UI (as global admin) and Mailu API as well.

@nextgens
Copy link
Contributor

Get your users to use tokens/application passwords. Those are exempt from rate limits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Enhances existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants