From 90975fe914d066726d06f53abe8696399b13a61a Mon Sep 17 00:00:00 2001
From: "justdave%syndicomm.com" <>
Date: Sat, 28 Sep 2002 11:10:27 +0000
Subject: Fix for bug 171296: changing Content-disposition header in
 attachment.cgi to use 'inline' instead of 'attachment' so that it doesn't
 *force* you to download it. r= bbaetz, bzbarsky

---
 attachment.cgi | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

(limited to 'attachment.cgi')

diff --git a/attachment.cgi b/attachment.cgi
index cfdbd4ea6..4d5fea475 100755
--- a/attachment.cgi
+++ b/attachment.cgi
@@ -342,17 +342,11 @@ sub view
     SendSQL("SELECT mimetype, filename, thedata FROM attachments WHERE attach_id = $::FORM{'id'}");
     my ($contenttype, $filename, $thedata) = FetchSQLData();
 
-    # Determine if the browser supports the Content-Disposition header or not
-    my $usedisposition = 1; # assume yes, unless we discover otherwise
-    if ($::ENV{HTTP_USER_AGENT} =~ /^Mozilla.*MSIE (\d+)/) {
-        if ($1 < 5) { $usedisposition = 0; } # MSIE < 5.0 chokes on it
-    }
-    
     # Return the appropriate HTTP response headers.
     $filename =~ s/^.*[\/\\]//;
     my $filesize = length($thedata);
     print qq{Content-Type: $contenttype; name="$filename"\n};
-    print qq{Content-Disposition: attachment; filename=$filename\n} if $usedisposition;
+    print qq{Content-Disposition: inline; filename=$filename\n};
     print qq{Content-Length: $filesize\n};
     print qq{\n$thedata};
 
-- 
cgit v1.2.3-24-g4f1b