Content-Disposition header parsing needs to strip out "filename="

VERIFIED FIXED in mozilla0.9.8

Status

P1
normal
VERIFIED FIXED
17 years ago
3 years ago

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Tracking

Trunk
mozilla0.9.8

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reviews?  It'd be nice not to ship 0.9.8 with that bug... :)
Status: NEW → ASSIGNED
Keywords: mozilla0.9.8
Priority: -- → P1
Target Milestone: --- → mozilla0.9.8
Created attachment 65344 [details] [diff] [review]
Use a variable for that string, per review comments
Attachment #65327 - Attachment is obsolete: true

Comment 5

17 years ago
Comment on attachment 65344 [details] [diff] [review]
Use a variable for that string, per review comments

r=sballard@netreach.net (bz said it would be okay for me to give r= without
actually testing for such a trivial patch; if that's not okay it's his fault ;)
)

Setting has-super-review flag based on sr for previous patch
Attachment #65344 - Flags: superreview+
Attachment #65344 - Flags: review+
a=brendan@mozilla.org for checkin to the frozen 0.9.8 trunk.

/be
Blocks: 115520
checked into trunk.  Thanks for the quick responses, all.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
tested by using Save Link As from the context menu for the "120409" link at the
top of this page. the resulting suggested filename is now "show_bug.cgi.html".
used 2002.01.29.xx comm verif bits on linux rh7.2, win2k and mac 10.1.2.

marking verified --but if this is still not the expected results, pls do reopen!
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.