The chat ads that are shown in the side bar of site pages incorrectly expect the user to always have a gravatar-based profile image:
var e = "http://www.gravatar.com/avatar/" + b.emailhash + "?s=23&d=identicon&r=PG"
This is incorrect behaviour for users who have Facebook or direct-upload profile images. As Szabolcs points out in chat, this is a particular problem for Facebook images which have their own query string, as it breaks the size argument to gravatar:
The JavaScript should check if the provided user emailhash
value is a URL, and use that in place of the constructed gravatar URL when appropriate.
For the related issue of the icon being allowed to be too large, the generated <img>
tag should be constrained to height="23" width="23"
just in case.
var e
shows the url for the image, whereb.emailhash
is the unique id (hashed email) for the user,s=23
is asking for a 23 pixel image andd=identicon
is what specifies the default image if no img exists (like in this case). They could choose to return a 404 in the case of no img (and do something else)... but apparently aren't. – MikeSmithDev Jan 18 '13 at 3:42b.emailhash
is an actual URL to the non-gravatar profile image in the above-mentioned cases, so you end up withhttp://www.gravatar.com/avatar/http://i.stack.imgur.com/ljxup.png?s=23&d=identicon&r=PG
. – Tim Stone Jan 18 '13 at 3:51