(cc-ing some area experts)
Hi Andrej,

Andrej Andb wrote:

> [Subject: remove protocol from gravatar and picon links for clear if
> Gitweb is being called through a secure server]

Sounds good to me.  May we have your signoff?  (See
Documentation/SubmittingPatches for what this means.)

Thanks,
Jonathan
(patch left unsnipped for reference)

> ---
>  gitweb/gitweb.perl | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index c6bafe6..1309196 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -2068,7 +2068,7 @@ sub picon_url {
>       if (!$avatar_cache{$email}) {
>               my ($user, $domain) = split('@', $email);
>               $avatar_cache{$email} =
> -                     
> "http://www.cs.indiana.edu/cgi-pub/kinzler/piconsearch.cgi/"; .
> +                     "//www.cs.indiana.edu/cgi-pub/kinzler/piconsearch.cgi/" 
> .
>                       "$domain/$user/" .
>                       "users+domains+unknown/up/single";
>       }
> @@ -2083,7 +2083,7 @@ sub gravatar_url {
>       my $email = lc shift;
>       my $size = shift;
>       $avatar_cache{$email} ||=
> -             "http://www.gravatar.com/avatar/"; .
> +             "//www.gravatar.com/avatar/" .
>                       Digest::MD5::md5_hex($email) . "?s=";
>       return $avatar_cache{$email} . $size;
>  }
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to