add support for LoginRequiredMiddleware with login_not_required decor…

django-commons / django-debug-toolbar

A configurable set of panels that display various debug information about the current request/response.

https://django-debug-toolbar.readthedocs.io

BSD 3-Clause "New" or "Revised" License

8.08k stars 1.05k forks source link

add support for LoginRequiredMiddleware with login_not_required decor… #2005

Closed dulmandakh closed 1 month ago

dulmandakh commented 1 month ago

Description

This PR adds support for LoginRequiredMiddleware introduced in Django 5.1, or views provided by this package don't require login.

Checklist:

[x] I have added the relevant tests for this change.
[x] I have added an item to the Pending section of docs/changes.rst.

dulmandakh commented 1 month ago

Thanks for working on this!

Here's an example of an app adding support for login_not_required while actually testing the behavior: matthiask/django-authlib@2239a1c

I contributed same support for Django OAuth Toolkit in https://github.com/jazzband/django-oauth-toolkit/pull/1454. Thanks for the reference.

dulmandakh commented 1 month ago

@matthiask please review and merge if this PR looks good

matthiask commented 1 month ago

Thanks!