From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason at zx2c4.com (Jason A. Donenfeld) Date: Wed, 8 Jan 2014 16:12:38 +0100 Subject: [RESEND PATCH 1/1] enable cgit to show gravatar for author, committer and tagger In-Reply-To: <1389192811-31573-1-git-send-email-mail@eworm.de> References: <1389192811-31573-1-git-send-email-mail@eworm.de> Message-ID: I like this. But I could imagine a bit of backlash against it -- it's a particular web service owned by a private corporation (Automattic), and some detractors might argue that an open project like this has no interest in this kind of proprietary service integration. Are there any such detractors on this list? If so, we can fill this thread with some sort of discussion. Otherwise, pending the nitpicks below, I'll merge it. On Wed, Jan 8, 2014 at 3:53 PM, Christian Hesse wrote: > + > +char * cgit_get_gravatar(const char *email) { > + unsigned char digest[MD5_DIGEST_LENGTH]; char *cgit_get_gravatar(const char *email) { > + /* The URL includes %s, which is replaced later on. So we do not need > + * extra space for termination. */ > + gravatar = malloc(strlen(GRAVATAR_URL) + MD5_DIGEST_LENGTH * 2); This is clever, but potentially error prone.