Closed Bug 675617 Opened 13 years ago Closed 13 years ago

Fix browser_installssl.js to support keep-alive connections.

Categories

(Testing :: General, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
mozilla8

People

(Reporter: mayhemer, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [inbound])

Attachments

(1 file)

Attached patch v1Splinter Review
As part of the work to make httpd.js support keep-alive connections this test needs a fix.

When reusing keep-alive connections while adding/removing/adding cert exceptions, the test suffers from bug 471995 and fails.  To work that around we need to force connection close for the .rdf file being queried, what is simply done just by adding ^headers^ file with Connection: close header.
Attachment #549782 - Flags: review?(dtownsend)
Attachment #549782 - Flags: review?(dtownsend) → review+
Is it possible to make the test not depend on the connection always being immediately and promptly closed?  That ^headers^ will include the Connection header (and various other transport-level-y headers) is arguably a bug, and it's possible at some point I'll do what XMLHttpRequest does and blacklist certain headers from being set there.
According the patches from the httpd.js keep-alive bug (at this moment unreviewed and unsettled) I would like to allow files say it wants to close the connection this way.  Also request handlers should be able to do it by adding the header.

The patch in this bug is mostly a workaround for bug 471995, which I want to start working on soon.  So the ^headers^ files will go away with fixing that bug.
http://hg.mozilla.org/mozilla-central/rev/efb4b265e268
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Component: httpd.js → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: