#36837: Client.force_login won't work for permission-only backends inheriting 
from
BaseBackend
-------------------------------------+-------------------------------------
     Reporter:  Christian Hartung    |                    Owner:  (none)
         Type:  New feature          |                   Status:  closed
    Component:  contrib.auth         |                  Version:  6.0
     Severity:  Normal               |               Resolution:
                                     |  needsnewfeatureprocess
     Keywords:                       |             Triage Stage:
                                     |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Comment (by Natgho):

 Replying to [comment:3 Jacob Walls]:
 > A tiny hint for the test client seems like a not disruptive change for
 the auth backends. It is new API, though, so we need a new features
 discussion to make sure we're not missing anything. It would also be nice
 to know if `django-rules` and `django-guardian` are willing to use the
 feature we build for them here.
 > ...
 > My recommendation is to kick off a [https://github.com/django/new-
 features new-features] discussion. Thanks.


 VIZZARD-X brought this ticket to our attention, and we’ve taken it into
 consideration. There is indeed an issue here regarding future
 compatibility, and he is putting in a serious effort to address it;

 https://github.com/django-guardian/django-guardian/pull/948

 I believe the second “tricky” solution you mentioned—adding a “noop” field
 and running tests through that field—would be an improvement that wouldn’t
 harm the core Django structure and would also easily ensure the tests run
 successfully.

 Speaking as a maintainer of the Django-Guardian team, if this small patch
 is accepted, we would be happy to use it and incorporate this feature into
 the project. Because the concept of asynchronous processing is a valuable
 feature we also need to adapt to. I’d like to note that we’re eager to use
 this feature if it’s added, to ensure your tests can be written and run
 successfully.
-- 
Ticket URL: <https://code.djangoproject.com/ticket/36837#comment:4>
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 visit 
https://groups.google.com/d/msgid/django-updates/0107019ce96b076c-88bdccc1-65f4-4afa-9289-ac1e5371d7e8-000000%40eu-central-1.amazonses.com.

Reply via email to