#34022: admin:logout fails to log out non-staff users
-------------------------------+--------------------------------------
     Reporter:  Jan Pazdziora  |                    Owner:  nobody
         Type:  New feature    |                   Status:  closed
    Component:  contrib.admin  |                  Version:  4.1
     Severity:  Normal         |               Resolution:  wontfix
     Keywords:                 |             Triage Stage:  Unreviewed
    Has patch:  0              |      Needs documentation:  0
  Needs tests:  0              |  Patch needs improvement:  0
Easy pickings:  0              |                    UI/UX:  0
-------------------------------+--------------------------------------
Changes (by Mariusz Felisiak):

 * status:  new => closed
 * type:  Uncategorized => New feature
 * resolution:   => wontfix


Comment:

 Thanks for the report, however the proposed change is backward
 incompatible and introduces a regression (see #159 and
 03eeb020a00dedba2594326bd606d8b41d51e80f). You should be able to use a
 custom `AdminSite` and overwrite `AdminSite.has_permission()`, e.g.
 {{{#!python
 class CustomAdminSite(AdminSite):
     def has_permission(self, request):
         return request.user.is_active
 }}}

-- 
Ticket URL: <https://code.djangoproject.com/ticket/34022#comment:2>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018354649825-2db14990-c272-4b51-b9b9-c578aaf36857-000000%40eu-central-1.amazonses.com.

Reply via email to