Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add ability to delete redis keys on demand #88

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

ecita
Copy link

@ecita ecita commented Mar 4, 2025

By submitting a PR to this repository, you agree to the terms within the Auth0 Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

We are faced with a usecase that requires us to delete specific redis key(s) on demand to manipulate the rate limiting behaviour.

This PR adds del function to limitd client interface in a manner that is aligned with the original Redis DEL function.
https://redis.io/docs/latest/commands/del/

Testing

  • This change adds test coverage for new/changed/fixed functionality

Checklist

  • I have added documentation for new/changed functionality in this PR or in auth0.com/docs
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used, if not the default branch

@ecita ecita requested a review from a team as a code owner March 4, 2025 13:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant