Skip to content
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

Jump to bottom

Require email verification when changing email address. #13533

Open
Findus23 opened this issue Oct 4, 2018 · 2 comments
Open

Require email verification when changing email address. #13533

Findus23 opened this issue Oct 4, 2018 · 2 comments
Labels
c: Security For issues that make Matomo more secure. Please report issues through HackerOne and not in Github. c: Usability For issues that let users achieve a defined goal more effectively or efficiently.
Milestone
For Prioritization

Comments

@Findus23
Copy link
Member

Findus23 commented Oct 4, 2018

Related to #2932 and #6125 (the latter could maybe also be added in the same change)

At the moment every Matomo user could change their E-Mail address to everything they want without any verification (apart from syntax, see #11796). This allows every Matomo user to send an unlimited amount of (for them) SPAM E-Mails to anyone without them ever opting in to receiving them.

When someone tries to change their email address (after confirming their password; #2932) the change should only be saved if the user was able to confirm an link in a sent mail.
In addition an email should be sent to the old email address informing them that they are loosing access to the account (and to inform an admin if they don't know about the change)
#6125
@Findus23 Findus23 added c: Security For issues that make Matomo more secure. Please report issues through HackerOne and not in Github. c: Usability For issues that let users achieve a defined goal more effectively or efficiently. labels Oct 4, 2018
@Findus23 Findus23 added this to the 3.8.0 milestone Oct 4, 2018
@tsteur
Copy link
Member

tsteur commented Oct 4, 2018

Slightly refs #13321 :)

@Findus23
Copy link
Member Author

Findus23 commented Oct 4, 2018

@tsteur The only issue I forgot to cross-link 🙂

@diosmosis diosmosis self-assigned this Feb 10, 2019
@diosmosis diosmosis mentioned this issue Feb 18, 2019
Closed
@mattab mattab removed this from the 3.9.0 milestone Feb 20, 2019
@diosmosis diosmosis mentioned this issue Feb 26, 2019
Merged
This was referenced May 24, 2019
Merged
Closed
@Findus23 Findus23 mentioned this issue Jun 5, 2019
Open
@diosmosis diosmosis removed their assignment Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: Security For issues that make Matomo more secure. Please report issues through HackerOne and not in Github. c: Usability For issues that let users achieve a defined goal more effectively or efficiently.
Projects
None yet
Milestone
For Prioritization
Development

No branches or pull requests
5 participants
@diosmosis @tsteur @mattab @Findus23 @innocraft-automation