Skip to content

Send confirmation when email address changed #998

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 · 5 comments
Closed

Send confirmation when email address changed #998

edmorley opened this issue Mar 3, 2016 · 5 comments

Comments

@edmorley
Copy link

edmorley commented Mar 3, 2016

Currently on PyPI, if a user changes their contact email address, there is no notification to the old email address, meaning account compromise can go unnoticed.

This means that an attacker can silently prevent other important security notifications (eg #997) from being sent to the real owner.

@brainwane
Copy link
Contributor

Thanks for another great suggestion, @edmorley, and sorry for the slow response!

This would be a very useful security feature, and now that #2898 is written and merged, it's far easier to add this feature.

Today in our development meeting we discussed where this should go on our development roadmap.

Since this feature isn't something that legacy PyPI has, I've moved the issue to a future milestone. But that doesn't have to stop a volunteer from working on it now; if someone wants to write up a pull request for it, go ahead!

Thanks and sorry again for the wait.

@waseem18
Copy link
Contributor

waseem18 commented Mar 6, 2018

I think we should send an email whenever an

  • email is added
  • email is deleted
  • email has been changed to primary email

@waseem18
Copy link
Contributor

waseem18 commented Mar 7, 2018

@brainwane I'm looking into this and will soon put up a PR.

@di
Copy link
Member

di commented Mar 7, 2018

@waseem18 We already send an email when a new address is added (the "verification" email). I don't think deleting an email should require a notification, most likely the address will be unused/invalid anyways, and a "primary change" action is the one with real security implications, so I think this issue is good. Thanks for your work!

@waseem18
Copy link
Contributor

waseem18 commented Mar 7, 2018

Your welcome @di :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants