Is this still being maintained? about django-ratelimit-backend HOT 5 OPEN

We learned a lot from that internal conversation! I'll link to our findings and subsequent code changes, as they may be useful to other users of this package:

• https://github.com/edx/edx-platform/blob/master/docs/decisions/0009_simplify_ratelimiting.rst
• openedx/edx-platform#26487

Basically, we found out that our login rate limiting in that service was a confusing pile of overlapping implementations. At least for our use case, django-ratelimit covers what we need (and was already in use for other URLs), so I'm afraid we'll be dropping our fork of this package. The notes above may help serve as a migration path for others.

from django-ratelimit-backend.

@jmbowman thanks, indeed I'm happy to transfer ownership to a maintained fork. Is the EDX fork the one you're referring to?

from django-ratelimit-backend.

Yes; I just kicked off an internal conversation about potentially adopting this, hoping to have a consensus on it by next week. That'll also give some time for anybody currently using another fork to chime in with any suggestions or questions.

from django-ratelimit-backend.

Thanks!

from django-ratelimit-backend.

Thanks @jmbowman, interesting read 👍

from django-ratelimit-backend.