From: Frédéric Buclin <LpSolit@gmail.com>
Date: Wed, 29 Apr 2015 09:41:00 +0000 (+0200)
Subject: Bug 622455: The attachment mime type autodetection shouldn't rely on the browser... 
X-Git-Tag: release-5.1.1~268
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=737ffe4a360ad028836522bddd658e234e71bbc7;p=thirdparty%2Fbugzilla.git

Bug 622455: The attachment mime type autodetection shouldn't rely on the browser when it gets text/x-*
r=dkl a=glob
---

diff --git a/Bugzilla/Attachment.pm b/Bugzilla/Attachment.pm
index ed770ad894..ed43cbe94d 100644
--- a/Bugzilla/Attachment.pm
+++ b/Bugzilla/Attachment.pm
@@ -541,9 +541,8 @@ sub _check_content_type {
 
     # If we have autodetected application/octet-stream from the Content-Type
     # header, let's have a better go using a sniffer if available.
-    if (defined Bugzilla->input_params->{contenttypemethod}
-        && Bugzilla->input_params->{contenttypemethod} eq 'autodetect'
-        && $content_type eq 'application/octet-stream'
+    if ((Bugzilla->input_params->{contenttypemethod} // '') eq 'autodetect'
+        && ($content_type eq 'application/octet-stream' || $content_type =~ m{text/x-})
         && Bugzilla->feature('typesniffer'))
     {
         import File::MimeInfo::Magic qw(mimetype);
@@ -574,8 +573,7 @@ sub _check_content_type {
 
     # Make sure patches are viewable in the browser
     if (!ref($invocant)
-        && defined Bugzilla->input_params->{contenttypemethod}
-        && Bugzilla->input_params->{contenttypemethod} eq 'autodetect'
+        && (Bugzilla->input_params->{contenttypemethod} // '') eq 'autodetect'
         && $content_type =~ m{text/x-(?:diff|patch)})
     {
         $params->{ispatch} = 1;