Skip to content

PyPI 2022-05-24: remove comment regarding email protections#34

Merged
ewdurbin merged 1 commit into
mainfrom
correct_factual_error
May 25, 2022
Merged

PyPI 2022-05-24: remove comment regarding email protections#34
ewdurbin merged 1 commit into
mainfrom
correct_factual_error

Conversation

@ewdurbin
Copy link
Copy Markdown
Collaborator

this paragraph is inaccurate, we do not require a verified email address to perform a password reset, see https://github.com/pypa/warehouse/blob/bd81ae8bae40fcd2736e47b65912db297a993339/warehouse/accounts/models.py#L165-L174 which is the only gate on password reset requests.

the email verification concerns noted in this paragraph are more to do with deliverability and reputation than security and would not have changed the course of this incident.

this paragraph is inaccurate, we do not require a verified email address to perform a password reset, see https://github.com/pypa/warehouse/blob/bd81ae8bae40fcd2736e47b65912db297a993339/warehouse/accounts/models.py#L165-L174 which is the only gate on password reset requests.

the email verification concerns noted in this paragraph are more to do with deliverability and reputation than security and would not have changed the course of this incident.
@ewdurbin ewdurbin merged commit 119e291 into main May 25, 2022
@ewdurbin ewdurbin deleted the correct_factual_error branch May 25, 2022 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant