jsocol/django-ratelimit

Alias function for is_ratelimited

Open

#308 aberto em 30 de out. de 2023

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Python (180 forks)batch import
help wanted

Métricas do repositório

Stars
 (904 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Hey,

Is_ratelimited is a great function, but the name is quite missleading for incrementing. You'd expect it only to be a boolean, but yet it can also increment. That's great, but I feel we should have also a new function like increment_ratelimit or some sort of increment function which is just a function rather than decorator, that allows you to increment whenever.

At the moment I want to only increment the ratelimit when the user does an unsuccessful login attempt. So i have

is_ratelimited(
                request, group="login_incorrect", key="ip", rate="5/5m", increment=True
)

But instead of that, i'd quite like something like:

increment_ratelimit(request, group="login_incorrect")

Instead i've created my own function that just does the same thing, but it'd be nice to see this in the official version.

Or I may have just missed it 🤷‍♂️

Happy to hear opinions, let me know if i'm just being blind :)

Guia do colaborador