From: mkanat%bugzilla.org <>
Date: Wed, 30 Sep 2009 08:51:45 +0000 (+0000)
Subject: Bug 328628: When attachments have UTF-8 characters in their name, they will now be... 
X-Git-Tag: bugzilla-3.5.1~52
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=dc4608b97cec3af3c7028717da291cd5e238a8d4;p=thirdparty%2Fbugzilla.git

Bug 328628: When attachments have UTF-8 characters in their name, they will now be downloaded with the correct name.
Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=Wurblzap, a=mkanat
---

diff --git a/attachment.cgi b/attachment.cgi
index 32f6e5ec01..2e952746e7 100755
--- a/attachment.cgi
+++ b/attachment.cgi
@@ -52,6 +52,8 @@ use Bugzilla::Attachment::PatchReader;
 use Bugzilla::Token;
 use Bugzilla::Keyword;
 
+use Encode qw(encode);
+
 # For most scripts we don't make $cgi and $template global variables. But
 # when preparing Bugzilla for mod_perl, this script used these
 # variables in so many subroutines that it was easier to just
@@ -318,6 +320,11 @@ sub view {
     $filename =~ s/\\/\\\\/g; # escape backslashes
     $filename =~ s/"/\\"/g; # escape quotes
 
+    # Avoid line wrapping done by Encode, which we don't need for HTTP
+    # headers. See discussion in bug 328628 for details.
+    local $Encode::Encoding{'MIME-Q'}->{'bpl'} = 10000;
+    $filename = encode('MIME-Q', $filename);
+
     my $disposition = Bugzilla->params->{'allow_attachment_display'} ? 'inline' : 'attachment';
 
     print $cgi->header(-type=>"$contenttype; name=\"$filename\"",