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

Trim and lowercase e-mails to conform with Gravatar guidelines #15

Merged
merged 2 commits into from Nov 14, 2017

Conversation

Projects
None yet
2 participants
@attis
Copy link
Contributor

attis commented Nov 14, 2017

https://gravatar.com/site/implement/hash/ says emails should be trimmed and lowercased before calculating hash. On our forums we have a situation where user with uppercase characters in his e-mail complained about Gravatar being missing even though old Forums displayed it. Hence this fix.

@julianlam julianlam merged commit 2c130bd into julianlam:master Nov 14, 2017

@julianlam

This comment has been minimized.

Copy link
Owner

julianlam commented Nov 14, 2017

v2.2.0, thanks!

@julianlam julianlam self-assigned this Nov 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.