Closed Bug 20383 Opened 20 years ago Closed 19 years ago

showattachment.cgi does not return filename

Categories

(Bugzilla :: Attachments & Requests, defect, P3)

x86
All
defect

Tracking

()

VERIFIED DUPLICATE of bug 63601

People

(Reporter: jasonab, Assigned: justdave)

References

Details

(Whiteboard: 2.16)

Attachments

(1 file)

When attempting to view or download an attachment, showattachment.cgi does not
return a filename, and instead returns its own name. This complicates matters
inordinantly when trying to save an attachment to disk, as you have no idea what
sort of file you are downloading, or what it should be called.
Status: NEW → ASSIGNED
The submitted patch uses an HTTP 1.1 header that Netscape and IE implement to
return the filename with the download. This does not affect text attachments,
however, and their name is still a cgi string.
Regarding "Remove any path information stored in the database"
in the attachment of 2000-02-07,
the regexp   /[^\\]+$/  assumes Windows pathnames.
To handle both Windows and Unix pathnames, try  m{[^/\\]+$} .
tara@tequilarista.org is the new owner of Bugzilla and Bonsai.  (For details,
see my posting in netscape.public.mozilla.webtools,
news://news.mozilla.org/38F5D90D.F40E8C1A%40geocast.com .)
Assignee: terry → tara
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
This bug has a patch on it waiting to get checked in...

Ah, but the patch is incomplete - assumes it's running on Windows.  but it's a 
three character change to get it running on both Windows and Linux and the change 
is described in an earlier comment here.
Keywords: patch
*** Bug 43286 has been marked as a duplicate of this bug. ***
Point of interest....   according to bug 6519, this functionality used to be in 
Bugzilla, and was removed on purpose because it broke the XML filetype.

What I would suggest here is that if this is put in, that it be conditional, and 
only add the name parameter if the file is known to be binary (since it's ignored 
on text, and breaks XML)
will look to evaluate
Assignee: tara → cyeh
Status: ASSIGNED → NEW
Whiteboard: 2.14
Whiteboard: 2.14 → 2.16
Adding default QA contact to all open Webtools/Bugzilla bugs lacking one.
Sorry for the spam.
QA Contact: matty
moving to real milestones...
Target Milestone: --- → Bugzilla 2.16
There is another bug about this subject which also has a patch attached, but
it looks more elaborated to me (and is in diff -u format), so I'm marking this
as a duplicate.

*** This bug has been marked as a duplicate of 63601 ***
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
verif. duplicate of 63601
Status: RESOLVED → VERIFIED
moving all closed Bugzilla bugs to the new Bugzilla product.
This batch is DUPLICATE/INVALID/WORKSFORME/WONTFIX
reassigning to default owner and QA in case of the bug being reopened.
Clearing milestones, since we really shouldn't have them on these types of
resolutions.  Sorry for the spam everyone...
Assignee: Chris.Yeh → justdave
Status: VERIFIED → NEW
Component: Bugzilla → Bugzilla-General
Product: Webtools → Bugzilla
Target Milestone: Bugzilla 2.16 → ---
Version: other → unspecified
Verification got lost.
Status: RESOLVED → VERIFIED
Component: Bugzilla-General → attachment and request management
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.