Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: mlk, perf
Assignee: neeti → darin
enabling pipelining has no effect currently, so this seems strange. but, if we are leaking sockets at all then this needs to be fixed with high priority. investigating...
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.1
I use no proxy but with 2001051813 linux i see heaps of http sockets remain open to sites i left long ago. On some sites it seems i overload the server while trying to browse a page, so it stops servinge me rest of content. Sample page that no longer completes loading is http://www.dinside.no/motor Clicking Stop button, or simply right-clicking in page (to invoke context menu) makes it load some more, but then it spins again. Looking at the sockets there are pretty many open to the site (and banner sites) - serving nothing anymore. There might be a related problem opening sockets as well. I frequently have to click "submit" button in bugzilla query form to make it send a request.
for me on win98, browser buster seems to be able to load only 15-20 urls before I start seeing 'connection refused' messages and attempts to view pages aborted. definite test blocker, and if users on some key platforms are only going to be able to use the browser for 15 or 20 sites then its a beta blocker.
Whiteboard: required for 0.9.1
changing OS to all... removing "pipelining" from summary.
OS: Windows 98 → All
Summary: Mozilla leaks sockets with HTTP pipelining. → Mozilla leaks HTTP sockets
the attached patch solves the problem by pruning out dead connections as new connections are requested. this does not prevent the browser from holding onto dead connections, but it does ensure that the number of dead connections will always tend to zero while browsing. we'd need some sort of timer event to properly clear out dead connections. but, with my patch at least the browser won't run out of sockets... servers may still dislike us for holding onto dead connections, though ;-) instituting the periodic cleanup of dead connections is a separate bug.
*** Bug 79967 has been marked as a duplicate of this bug. ***
dougt says: 'sr='
fix checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
looking forward to this fix... I still see the problem in build from this morning but doesn't seem to be as bad... I'm I just hitting the right side of randomness?
try a build from 5/23
VERIFIED with comm branch 2001060713 linux build. Not marking this as verified until other platforms have been checked.
Verified. The host URL no longer exists. Reopen if this bug still exists.
Status: RESOLVED → VERIFIED
QA Contact: tever → junruh
You need to log in before you can comment on or make changes to this bug.