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

Auth/Mailer: properly handle usernames including a comma #2504

Open
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@LarsGit223
Collaborator

LarsGit223 commented Sep 15, 2018

Prevent splitting of e-mail addresses at the wrong point by enclosing a username in '"'. The "To" e-mail address in the notification mail was malformed if a new user was added and included a ',' in it's name. Fixes #1569.

LarsGit223 added some commits Sep 15, 2018

Auth/Mailer: properly handle usernames including a comma
Prevent splitting of e-mail addresses at the wrong point by enclosing a username in '"'.
The "To" e-mail address in the notification mail was malformed if a new user was added
and included a ',' in it's name. Fixes #1569.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment