Jakub Narębski <jna...@gmail.com> writes:

> Jürgen Kreileder wrote:
>
>> Properly encode site and project names for RSS and Atom feeds.
>> 
>> Signed-off-by: Jürgen Kreileder <j...@blackdown.de>
>> ---
>>  gitweb/gitweb.perl |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
>> index 9cfe5b5..09294eb 100755
>> --- a/gitweb/gitweb.perl
>> +++ b/gitweb/gitweb.perl
>> @@ -8056,7 +8056,7 @@ sub git_feed {
>>      return if ($cgi->request_method() eq 'HEAD');
>>  
>>      # header variables
>> -    my $title = "$site_name - $project/$action";
>> +    my $title = to_utf8($site_name) . " - " . to_utf8($project) . 
>> "/$action";
>>      my $feed_type = 'log';
>>      if (defined $hash) {
>>              $title .= " - '$hash'";
>> 
>
> Was this patch triggered by some bug?

Yes, I actually see broken encoding with the old code, e.g on 
https://git.blackdown.de/old.cgi?p=contactalbum.git;a=rss
my first name is messed up in the title tag.

New version: https://git.blackdown.de/?p=contactalbum.git;a=rss

> Because the above is not necessary, as git_feed() has
>
>       $title = esc_html($title);
>
> a bit later, which does to_utf8() internally.

Good point.  But it doesn't fix the string in question:
It looks like to_utf8("$a $b") != (to_utf8($a) . " " . to_utf($b)).


Juergen
--
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