-
Bug
-
Resolution: Fixed
-
Low (View bug fix roadmap)
-
5.1
-
None
-
5.01
-
2
-
Gravatar implementation in com.atlassian.jira.avatar.AvatarServiceImpl.GravatarUrlStrategy doesn't convert email address to lower case.
According to http://en.gravatar.com/site/implement/hash/ it should have been lowercased.
Haven't tested it but this patch should fix it.
--- AvatarServiceImpl.java Thu Jul 5 03:30:04 2012
+++ AvatarServiceImpl2.java Wed Jul 11 18:42:44 2012
@@ -370,7 +370,7 @@
if (user != null && user.getEmailAddress() != null)
{
String email = user.getEmailAddress();
- String hash = MD5Util.md5Hex(email);
+ String hash = MD5Util.md5Hex(email.toLowerCase());
URI jiraBaseUrl = URI.create(velocityRequestContextFactory.getJiraVelocityRequestContext().getCanonicalBaseUrl());
String gravatarBaseUrl;
- is duplicated by
-
-
- Closed
-
- mentioned in
-
![[Extranet] Page](/images/icons/generic_link_16.png "[Extranet] Page")
Wiki Page
Failed to load
Gravatar integration should use lower case email addresses
-
-
Resolution: Fixed
-
Low
-
5.1
-
None
-
5.01
-
2
-
Gravatar implementation in com.atlassian.jira.avatar.AvatarServiceImpl.GravatarUrlStrategy doesn't convert email address to lower case.
According to http://en.gravatar.com/site/implement/hash/ it should have been lowercased.
Haven't tested it but this patch should fix it.
--- AvatarServiceImpl.java Thu Jul 5 03:30:04 2012
+++ AvatarServiceImpl2.java Wed Jul 11 18:42:44 2012
@@ -370,7 +370,7 @@
if (user != null && user.getEmailAddress() != null)
{
String email = user.getEmailAddress();
- String hash = MD5Util.md5Hex(email);
+ String hash = MD5Util.md5Hex(email.toLowerCase());
URI jiraBaseUrl = URI.create(velocityRequestContextFactory.getJiraVelocityRequestContext().getCanonicalBaseUrl());
String gravatarBaseUrl;
- is duplicated by
-
-
- Closed
-
- mentioned in
-
Wiki Page Loading...
-
Eric Dalgliesh
-
- Votes:
-
5 Vote for this issue
- Watchers:
-
9 Start watching this issue
- Created:
- Updated:
- Resolved: