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

feat(rate-limiting): implement IP-based rate limiting and optimize request thresholds #3

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

Conversation

tgrecojs
Copy link
Contributor

@tgrecojs tgrecojs commented Feb 5, 2025

  • Add IP-based rate limiting using Redis with sliding window
  • Increase global request limit from 15 to 50 requests per 15s window
  • Adjust IP limit to 30 requests per minute
  • Reduce processing delay from 100ms to 50ms for better throughput
  • Add rate limit metrics tracking
  • Update error handling for rate limit exceeded cases

This change improves API protection against abuse while maintaining good UX for legitimate users.

…quest thresholds

- Add IP-based rate limiting using Redis with sliding window
- Increase global request limit from 15 to 50 requests per 15s window
- Adjust IP limit to 30 requests per minute
- Reduce processing delay from 100ms to 50ms for better throughput
- Add rate limit metrics tracking
- Update error handling for rate limit exceeded cases

This change improves API protection against abuse while maintaining good UX for legitimate users.
@tgrecojs tgrecojs added the enhancement New feature or request label Feb 5, 2025
@tgrecojs tgrecojs requested a review from zmanian February 5, 2025 05:07
@tgrecojs tgrecojs self-assigned this Feb 5, 2025
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
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant