This is not a problem when using the default set of common passwords, as
they have been preprocessed to be lowercase. However, there is nothing in
the documentation indicating this preprocessing should occur.
--
Ticket URL: <https://code.djangoproject.com/ticket/28881>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Old description:
> The CommonPasswordValidator holds a set of common passwords in memory,
> after strip()ing any whitespace.
> While validating a password, it converts it to lowercase before comparing
> to the set. However, the reference set was not converted to lowercase.
>
> This is not a problem when using the default set of common passwords, as
> they have been preprocessed to be lowercase. However, there is nothing in
> the documentation indicating this preprocessing should occur.
New description:
The CommonPasswordValidator holds a set of common passwords in memory,
after strip()ing any whitespace.
While validating a password, it converts it to lowercase before comparing
to the set. However, the reference set was not converted to lowercase.
This is not a problem when using the default set of common passwords, as
they have been preprocessed to be lowercase. However, there is nothing in
the documentation indicating this preprocessing should occur.
https://github.com/django/django/pull/9414
--
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:1>
Old description:
> The CommonPasswordValidator holds a set of common passwords in memory,
> after strip()ing any whitespace.
> While validating a password, it converts it to lowercase before comparing
> to the set. However, the reference set was not converted to lowercase.
>
> This is not a problem when using the default set of common passwords, as
> they have been preprocessed to be lowercase. However, there is nothing in
> the documentation indicating this preprocessing should occur.
>
> https://github.com/django/django/pull/9414
New description:
The CommonPasswordValidator holds a set of common passwords in memory,
after strip()ing any whitespace.
While validating a password, it converts it to lowercase before comparing
to the set. However, the reference set was not converted to lowercase.
This is not a problem when using the default set of common passwords, as
they have been preprocessed to be lowercase. However, there is nothing in
the documentation indicating this preprocessing should occur.
--
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:2>
* version: 1.11 => 2.0
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:3>
* component: Uncategorized => Documentation
* type: Uncategorized => Cleanup/optimization
* stage: Unreviewed => Accepted
Comment:
I think documenting the existing requirement for the password list to be
lower case would be fine. A documentation fix would address the problem
for older versions of Django and also avoid unnecessary computations for
lists that are already lower case.
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:4>
* owner: nobody => Karmen
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:5>
* has_patch: 0 => 1
* stage: Accepted => Ready for checkin
Comment:
[https://github.com/django/django/pull/9586#pullrequestreview-88743187 PR]
adds small doc fix.
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:6>
Comment (by Tim Graham <timograham@…>):
In [changeset:"146317b734275daddcc9b5e240b3c48b508fc6e6" 146317b7]:
{{{
#!CommitTicketReference repository=""
revision="146317b734275daddcc9b5e240b3c48b508fc6e6"
[2.0.x] Fixed #28881 -- Doc'd that CommonPasswordValidator's password list
must be lowercase.
Backport of 4fcd28d442c2fec56f544f99cb658f33f847824c from master
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:8>
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"4fcd28d442c2fec56f544f99cb658f33f847824c" 4fcd28d]:
{{{
#!CommitTicketReference repository=""
revision="4fcd28d442c2fec56f544f99cb658f33f847824c"
Fixed #28881 -- Doc'd that CommonPasswordValidator's password list must be
lowercase.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/28881#comment:7>