From 7a5013325ce52f25638de1a9c570ecd8fbe7e9a9 Mon Sep 17 00:00:00 2001 From: Damian Bushong Date: Fri, 5 Jul 2013 16:02:59 -0500 Subject: [PATCH] Force a default gravatar if no email provided Gravatar provides the forceDefault option, let's use it. --- src/user.js | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/user.js b/src/user.js index c2fec6851e..d0ce0a7e9a 100644 --- a/src/user.js +++ b/src/user.js @@ -224,13 +224,14 @@ var utils = require('./../public/src/utils.js'), }; User.createGravatarURLFromEmail = function(email) { + var forceDefault = '' if (!email) { - email = utils.generateUUID(); + email = '0000', + forceDefault = '&forceDefault=y'; } - var md5sum = crypto.createHash('md5'); - md5sum.update(email.toLowerCase().trim()); - var gravatarURL = 'http://www.gravatar.com/avatar/' + md5sum.digest('hex') + '?default=identicon&s=128'; - return gravatarURL; + var emailHash = crypto.createHash('md5').update(email.toLowerCase().trim()).digest('hex'); + // @todo: https asset support? + return 'http://www.gravatar.com/avatar/' + emailHash + '?default=identicon&s=128' + forceDefault; } User.hashPassword = function(password, callback) {