Fix for bug 111522: Provide ability to specify MIME type of attachment

when downloading. Adds a 'ctype' argument to attachment.cgi which allows one to override the content-type when viewing an attachment. Original patch by Alex Vincent <ajvincent@juno.com>; I changed it a tiny bit. r=kiko, myk. a=justdave.

Fix for bug 111522: Provide ability to specify MIME type of attachment
376c76bd · kiko%async.com.br · 77c68c2b · 376c76bd
Commit 376c76bd authored Oct 29, 2003 by kiko%async.com.br
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 0 deletions

attachment.cgi attachment.cgi +10 -0

No files found.
--- a/attachment.cgi
+++ b/attachment.cgi
@@ -22,6 +22,7 @@
 #                 Myk Melez <myk@mozilla.org>
 #                 Daniel Raichle <draichle@gmx.net>
 #                 Dave Miller <justdave@syndicomm.com>
+#                 Alexander J. Vincent <ajvincent@juno.com>

 ################################################################################
 # Script Initialization
@@ -433,6 +434,15 @@ sub view
    # Retrieve the attachment content and its content type from the database.
    SendSQL("SELECT mimetype, filename, thedata FROM attachments WHERE attach_id = $::FORM{'id'}");
    my ($contenttype, $filename, $thedata) = FetchSQLData();
+   
+    # Bug 111522: allow overriding content-type manually in the posted $::FORM.
+    if ($::FORM{'ctype'})
+    {
+        $::FORM{'contenttypemethod'} = 'manual';
+        $::FORM{'contenttypeentry'} = $::FORM{'ctype'};
+        validateContentType();
+        $contenttype = $::FORM{'ctype'};
+    }

    # Return the appropriate HTTP response headers.
    $filename =~ s/^.*[\/\\]//;