Closed
Bug 83304
Opened 24 years ago
Closed 24 years ago
download dialog comes up for image/jpg content type url
Categories
(Core :: Graphics: ImageLib, defect)
Tracking
()
VERIFIED
FIXED
mozilla0.9.2
People
(Reporter: blizzard, Assigned: pavlov)
References
()
Details
(Whiteboard: r=pinkerton, sr=blizzard, a=chofmann PDT+,critical for 0.9.2, ETA: 6/21/2001)
Attachments
(1 file)
|
980 bytes,
patch
|
Details | Diff | Splinter Review |
Build is from the morning of May 30, 2001 after the recent content-type header fix.
Loading the url brings up a download dialog asking if the user wants to save
type image/jpg type. It should just show it. The headers from the server look
like this:
[blizzard@face blizzard]$ telnet www.intellicast.com 80
Trying 208.194.150.10...
Connected to www.intellicast.com.
Escape character is '^]'.
HEAD /WeatherImages/CurrentTemps/cast.wsi;usa;CurrentTemps;jpg;CurrentTemps.jpg
HTTP/1.0
HTTP/1.1 200 OK
Server: Netscape-Enterprise/4.0
Date: Wed, 30 May 2001 16:08:10 GMT
Expires: Wed, 30 May 2001 16:08:10 GMT
Expires: Wed, 30 May 2001 16:08:10 GMT
Last-modified: Wed, 30 May 2001 16:08:10 GMT
Content-type: image/jpg
Accept-ranges: bytes
Connection: close
And it actually dumps the body of the image out even though it's a HEAD request.
Yay, lame CGIs! Anyway, I'm filing this against imglib but I suspect it's not
actually imglib but a networking thing which is why all these people are cc'ed.
|
||
Comment 1•24 years ago
|
||
Well, image/jpg is the wrong mime type... (the right one is image/jpeg).
See http://lxr.mozilla.org/seamonkey/source/content/build/nsContentDLF.cpp#81
and http://lxr.mozilla.org/seamonkey/source/layout/build/nsLayoutDLF.cpp#96 for
obvious places where changes would need to be made to support image/jpg
|
Assignee | |
Comment 4•24 years ago
|
||
I need to add some code to map things like image/pjpeg and image/jpg to
image/jpeg.
Target Milestone: --- → mozilla1.0
|
Reporter | |
Updated•24 years ago
|
Whiteboard: critical for 0.9.2
|
||
Updated•24 years ago
|
Whiteboard: critical for 0.9.2 → PDT+,critical for 0.9.2
|
|
||
Comment 5•24 years ago
|
||
pav, got an ETA for the fix?
Whiteboard: PDT+,critical for 0.9.2 → PDT+,critical for 0.9.2, need ETA
|
|
Assignee | |
Updated•24 years ago
|
Target Milestone: mozilla1.0 → mozilla0.9.2
|
|
Assignee | |
Comment 6•24 years ago
|
||
the url in this bug doesn't seem to cause this problem anymore.
|
|
Assignee | |
Comment 7•24 years ago
|
||
|
|
Assignee | |
Updated•24 years ago
|
Keywords: patch
Whiteboard: PDT+,critical for 0.9.2, need ETA → need r=, sr=, a= PDT+,critical for 0.9.2, ETA: 6/21/2001
|
||
Comment 8•24 years ago
|
||
r=pink
|
|
Reporter | |
Comment 9•24 years ago
|
||
sr=blizzard
|
|
||
Comment 10•24 years ago
|
||
a=chofmann
|
|
Assignee | |
Comment 11•24 years ago
|
||
fixed
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Whiteboard: need r=, sr=, a= PDT+,critical for 0.9.2, ETA: 6/21/2001 → r=pinkerton, sr=blizzard, a=chofmann PDT+,critical for 0.9.2, ETA: 6/21/2001
|
||
Comment 12•24 years ago
|
||
this URL works now and verified fix checked into lxr.mozilla.org
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•