-
Notifications
You must be signed in to change notification settings - Fork 492
Open
Description
Problem
The ratelimit package currently depends on mediocregopher/radix v3, which does not support context-aware operations.
As a result, when a context is canceled (e.g., due to client timeouts), ratelimit may still:
- Wait for a connection from the Redis connection pool
- Execute Redis commands after the request has been canceled
This leads to unnecessary resource usage and potential latency under load.
Proposal
Upgrade to mediocregopher/radix v4, which introduces support for context.Context in command execution.
Benefits
- Proper context cancellation handling
- Reduced resource usage when requests are aborted
- Better integration with modern Go context patterns
Metadata
Metadata
Assignees
Labels
No labels