Skip to content

Send confirmation email when password changed #999

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
edmorley opened this issue Mar 3, 2016 · 7 comments · Fixed by #2959
Closed

Send confirmation email when password changed #999

edmorley opened this issue Mar 3, 2016 · 7 comments · Fixed by #2959
Assignees

Comments

@edmorley
Copy link

edmorley commented Mar 3, 2016

Currently on PyPI, if a user changes their password, there is no email notification, meaning account compromise can go unnoticed.

Changing of password doesn't seem to be implemented on https://warehouse.python.org/ yet, but filing this now to ensure it's not forgotten :-)

@edmorley
Copy link
Author

edmorley commented Mar 3, 2016

(And for that matter, I really hope the new implementation will require that the user confirm their existing password, rather than just allow changing it via the session cookie.)

@brainwane
Copy link
Contributor

@edmorley Thanks for creating this issue, and sorry for the slow response!

The folks working on Warehouse have gotten funding to concentrate on improving and deploying Warehouse, and have kicked off work towards our development roadmap -- the most urgent task is to improve Warehouse to the point where we can redirect pypi.python.org to pypi.org so the site is more sustainable and reliable.

We're working on #2831 right now, which should also address this issue.

Thanks and sorry again for the wait.

@waseem18
Copy link
Contributor

waseem18 commented Feb 12, 2018

@di This looks like an issue I can work on (no assignee 😄 )

Will start working on this.

@di
Copy link
Member

di commented Feb 12, 2018

Ah sorry again @waseem18, this issue is going to get closed along with #2831 in the same PR.

@di di self-assigned this Feb 12, 2018
@waseem18
Copy link
Contributor

waseem18 commented Feb 12, 2018

My bad! Can you please link me to an issue , I would be happy to work on it?

Thanks

@di
Copy link
Member

di commented Feb 12, 2018

@waseem18 #1886 might be a good one for you. Feel free to ask any questions on that issue.

@waseem18
Copy link
Contributor

Thanks for the suggestion @di :)

I'll hop on to it.

@di di mentioned this issue Feb 16, 2018
@di di closed this as completed in #2959 Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants