Opened 7 days ago
Last modified 4 days ago
#36226 assigned Bug
Only PBKDF2PasswordHasher supports str and bytes password
Reported by: | Jason Held | Owned by: | Screamadelica |
---|---|---|---|
Component: | contrib.auth | Version: | 5.1 |
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
The other password hashers in their encode
method all at some point call .encode
on their password in some way. Thus, the other hasher classes, in the django source code, do not support the API and docstring of make_password
.
This seems pretty easy to fix on the whole and I'd be happy to make a PR for it.
Change History (6)
comment:1 by , 7 days ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:2 by , 5 days ago
Owner: | set to |
---|---|
Status: | new → assigned |
Hi, I'm new to this community and find this a decent good first issue. I will try to fix this bug :)
comment:3 by , 5 days ago
Has patch: | set |
---|
follow-up: 5 comment:4 by , 5 days ago
Just finished a pr and all checks have passed.
https://github.com/django/django/pull/19231
follow-up: 6 comment:5 by , 4 days ago
Replying to Screamadelica:
Just finished a pr and all checks have passed.
https://github.com/django/django/pull/19231
It looks like you've submitted a PR, so I’ve set the "has patch" flag. Now, all that’s left is to wait for reviews from the fellows :)
comment:6 by , 4 days ago
Replying to Antoliny:
Replying to Screamadelica:
Just finished a pr and all checks have passed.
https://github.com/django/django/pull/19231
It looks like you've submitted a PR, so I’ve set the "has patch" flag. Now, all that’s left is to wait for reviews from the fellows :)
Thanks a lot, I will remember to change the flag after submitting PR later :)
Replicated thank you! PRs welcome
tests/auth_tests/test_hashers.py