Closed Bug 96146 Opened 23 years ago Closed 4 months ago

Multipart pages (like buglist.cgi) not saved in any usable form

Categories

(Firefox :: File Handling, defect, P5)

x86
Linux
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: greenrd, Unassigned)

References

()

Details

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.3+) Gecko/20010817
BuildID:    2001082012

Pages with content-disposition: inline, like bugzilla buglists, get saved as
MIME, not HTML.

The average user expects an ordinary webpage to save as HTML.

Reproducible: Always
Steps to Reproduce:
1. Go to URL above
2. Save page
3. View saved page with text editor

Actual Results:  Saved page starts with:

--thisrandomstring
Content-type: text/html
 
<html><head><title>Bugzilla is pondering your query</title>
<style type="text/css">
    .psb { margin-top: 20%; text-align: center; }
</style></head><body>
<h1 class="psb">Please stand by ...</h1></body></html>
 
--thisrandomstring
Content-type: text/html
Content-disposition: inline; filename=bugzilla_bug_list.html
Set-Cookie: LASTORDER=bugs.bug_id ; path=/; expires=Sun, 30-Jun-2029 00:00:00 GMT

Expected Results:  Saved only the last part (current page) as HTML.
-browser general - probably not file. xp-apps?
Assignee: dougt → asa
Component: Networking: File → Browser-General
QA Contact: benc → doronr
->xp apps it is.
Assignee: asa → pchen
Status: UNCONFIRMED → NEW
Component: Browser-General → XP Apps
Ever confirmed: true
QA Contact: doronr → sairuh
-> law
Assignee: pchen → law
Target Milestone: --- → mozilla1.1
spam: over to File Handling.
Component: XP Apps → File Handling
Summary: Pages with content-disposition: inline not saved as HTML → Multipart pages not saved as HTML
*** Bug 120187 has been marked as a duplicate of this bug. ***
Just to clarify the expected results from the original comment:

We should save the current document of the server-push stream, just as we would
with client-pull (http refresh).

This would allow the "Please stand by ..." bit to be saved, (as well as
view-source'd, but that probably needs a seperate bug).

Summary: Multipart pages not saved as HTML → Multipart pages not saved in any usable form
Adding keywords from dupe, adding most common case to the summary.
Keywords: nsbeta1
Summary: Multipart pages not saved in any usable form → Multipart pages (like buglist.cgi) not saved in any usable form
nsbeta1- per Nav triage team.  If anyone can show reproducible case on a top 500
site, please renominate.
Keywords: nsbeta1nsbeta1-
I would actually be tempted to save each part of the multipart stream in a
separate file... (nontrivial when they have different types, unfortunately).
*** Bug 160321 has been marked as a duplicate of this bug. ***
would there be time for this in buffy?
Keywords: nsbeta1-nsbeta1
QA: per comment 8, is this an issue for top sites?  For example, do travel sites
send back multipart pages when giving users confirmation that they may intend to
save for future reference?
Keywords: qawanted
Whiteboard: [need info]
QA Contact: sairuh → petersen
Nav triage team: nsbeta1-
Keywords: nsbeta1nsbeta1-
Whiteboard: [need info]
*** Bug 192050 has been marked as a duplicate of this bug. ***
*** Bug 208646 has been marked as a duplicate of this bug. ***
retargeting
Target Milestone: mozilla1.1alpha → Future
.
Assignee: law → file.handling
Priority: -- → P5
Keywords: qawanted
QA Contact: chrispetersen → ian
Assignee: file-handling → nobody
QA Contact: ian → file-handling
Product: Core → Firefox
Target Milestone: Future → ---
Version: Trunk → unspecified

Marking this as Resolved > Worksforme since the issue doesn't happen on Ubuntu 20.04 on either of the Firefox versions.
If anyone is still able to reproduce this please re-open or file a new issue.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WORKSFORME

Marking this as Resolved > Worksforme since the issue doesn't happen on Ubuntu 20.04 on either of the Firefox versions.

How did you test? Note that the URL in the URL field no longer sends a multipart responses last I checked, so it's not a useful testscase for this bug.

Flags: needinfo?(andrei.purice)

I tested with a "bugzilla buglists" page where I saved it and it was saved as HTML and not MIME.
If you can repro this Boris we would appreciate the input.

Flags: needinfo?(andrei.purice)

I tested with a "bugzilla buglists" page

My point is that this page no longer uses multipart responses last I checked. Have you tested with an actual multipart page? If not, please set up a web server that serves one and test with that.

Flags: needinfo?(andrei.purice)

I did not know it doesn't use multipart any longer.
Installing a web server is beyond my knowledge and expertise.
If you know how to test this more thoroughly could you give me a hand and check if you can reproduce this issue?

Flags: needinfo?(andrei.purice) → needinfo?(bzbarsky)

Installing a web server is beyond my knowledge and expertise.

Can you find someone else on your team to look at this, then?

Running a local server is easy (e.g. see https://pythonbasics.org/webserver/) but writing the actual testcase is a bit more time, and I just can't devote any more time to this at this point. I've already spent several hours this week addressing needinfo requests, for something that is very much no longer my job.

Flags: needinfo?(bzbarsky) → needinfo?(andrei.purice)

@Andrei: Can you then please revert your change to the status, if you can't test it ?

I started my web server but the multipart form data page is something I havent done before, Ill do a bit of research on this and try to reproduce this issue as soon as I have more time.
Will come back with positive results hopefully.
Re-opening the issue and will retest as soon as we get the Multipart Form data page going.

Status: RESOLVED → REOPENED
Flags: needinfo?(andrei.purice)
Resolution: WORKSFORME → ---

In the process of migrating remaining bugs to the new severity system, the severity for this bug cannot be automatically determined. Please retriage this bug using the new severity system.

Severity: major → --

The severity field is not set for this bug.
:Gijs, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(gijskruitbosch+bugs)

Reproduced with the testcase at the top of bug 1276918. However, given that bug and its recent activity, I believe this is probably wontfix.

Status: REOPENED → RESOLVED
Closed: 3 years ago4 months ago
Flags: needinfo?(gijskruitbosch+bugs)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.