#24855 closed New feature (fixed)
Allow the use of django.contrib.auth.login without credentials (authentication)
| Reported by: | Paulo Gabriel Poiati | Owned by: | Paulo Gabriel Poiati |
|---|---|---|---|
| Component: | contrib.auth | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
As it is we need to call authenticate before login because we need to set the authentication backend in the user model. We can use login without authenticate if we set the backend attr manually, but this need some implementation knowledge of how authentication works.
The proposal is to change the login function signature to:
login(request, user, backend=None)
The following rules will apply:
- Use the value of the
backendargument, if provided. - Use the value of the
user.backendannotation, if present. - Use the only configured backend, if there is only one.
- Raise ValueError("You have multiple authentication backends configured; you must provide the backend argument to login.") otherwise.
More information can be found in the mailing list topic:
https://groups.google.com/forum/#!topic/django-developers/_I0BW5L7Z58
Change History (11)
comment:1 by , 9 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 9 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:3 by , 9 years ago
comment:4 by , 9 years ago
| Has patch: | set |
|---|
follow-up: 6 comment:5 by , 9 years ago
| Needs documentation: | set |
|---|
comment:7 by , 9 years ago
| Needs documentation: | unset |
|---|
comment:8 by , 9 years ago
| Patch needs improvement: | set |
|---|
Note:
See TracTickets
for help on using tickets.
Patch: https://github.com/django/django/pull/5370