From 9907f86ebd19706ce79b2b0cf2f54e8ae4112427 Mon Sep 17 00:00:00 2001 From: Don Armstrong Date: Tue, 29 Oct 2013 13:56:53 -0700 Subject: [PATCH] uri_escape e-mails in libravatar (closes: #728034). Thanks to Fabien Givors for pointing this out and providing a patch. --- Debbugs/CGI/Bugreport.pm | 5 +++-- debian/changelog | 2 ++ 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/Debbugs/CGI/Bugreport.pm b/Debbugs/CGI/Bugreport.pm index 2e53892..67ed7ba 100644 --- a/Debbugs/CGI/Bugreport.pm +++ b/Debbugs/CGI/Bugreport.pm @@ -41,6 +41,7 @@ use Debbugs::UTF8; use Debbugs::Config qw(:config); use POSIX qw(strftime); use Encode qw(decode_utf8 encode_utf8); +use URI::Escape qw(uri_escape); BEGIN{ ($VERSION) = q$Revision: 494 $ =~ /^Revision:\s+([^\s+])/; @@ -148,7 +149,7 @@ sub display_entity { if ($_ eq 'From' and $param{avatars}) { my $libravatar_url = __libravatar_url(decode_rfc1522($head_field)); if (defined $libravatar_url and length $libravatar_url) { - push @headers,q(\n); + push @headers,q(\n); } } push @headers, qq(
$_: ) . html_escape(decode_rfc1522($head_field))."
\n"; @@ -452,7 +453,7 @@ sub __libravatar_url { return undef; } ($email) = get_addresses($email); - return $config{libravatar_uri}.$email.($config{libravatar_uri_options}//''); + return $config{libravatar_uri}.uri_escape($email.($config{libravatar_uri_options}//'')); } diff --git a/debian/changelog b/debian/changelog index e2ab47d..7a8bc94 100644 --- a/debian/changelog +++ b/debian/changelog @@ -18,6 +18,8 @@ debbugs (2.4.2~exp2) UNRELEASED; urgency=low (closes: #670456). Thanks to Jonathan Nieder and Bernhard Schmidt. * Add libravatar support. * Fix double encoding of attachments (closes: #703300) + * uri_escape e-mails in libravatar (closes: #728034). Thanks to Fabien + Givors for pointing this out and providing a patch. [Thanks to Arnout Engelen: ] * Add Homepage (closes: #670555). -- 2.39.2