Skip to content

Conversation

k9ert
Copy link

@k9ert k9ert commented Jul 29, 2025

If a user tries to login with his phone-number too often, he gets blocked via an entry in redis of the form: login_attempt_id:+192828282

However if the user asks in support to get unblocked, this is currently not yet possible. We'll need an admin-panel PR for that but we also need a admin-api endpoint which enables that.

So this PR:

  • Creates two admin-endpoints to search and remove limiter based redis-keys (and ONLY limiter based redis keys)
  • Adds two scripts to easily get access to the local dev admin-api
  • adds the redis cli to flake.nix
  • add some tests
# For unit tests
buck2 run //core/api:unit-tests -- --testPathPattern="reset-rate-limit"

# For integration tests  
buck2 run //core/api:test-integration -- --testPathPattern="reset-rate-limit"

@github-actions github-actions bot added the core label Jul 29, 2025
@k9ert k9ert requested a review from dolcalmi July 29, 2025 10:57
Copy link

@dolcalmi dolcalmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check this keybase://chat/blinkbtc#dev/1819

@k9ert k9ert changed the title feat: search and remove redis keys feat: search and reset redis limiter keys Jul 29, 2025
@openoms openoms marked this pull request as draft August 6, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants