Closed Bug 913827 Opened 8 years ago Closed 6 years ago

HTTP cache v2: let FTP use the new cache or drop cache use in it completely

Categories

(Core :: Networking: Cache, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED

People

(Reporter: mayhemer, Assigned: dragana)

References

Details

Attachments

(1 file)

To makes things simpler, I vote for just dropping it completely.
Blocks: 913828
We don't care at all about FTP performance (and IIRC we were caching only for directory listings, and browsing FTP that way is even rarer than clicking on an FTP download). So yes, let's disable cache usage.
Blocks: 1061898
No longer blocks: 1061898
Where's the FTP stuff living?
(In reply to Florian Bender from comment #2)
> Where's the FTP stuff living?

netwerk/protocol/ftp
Assignee: nobody → dd.mozilla
Attachment #8570192 - Flags: review?(michal.novotny)
Attachment #8570192 - Flags: review?(michal.novotny) → review+
Keywords: checkin-needed
That patch didn't have the bug number in the commit message, and I somehow managed to mistype it when I added it myself.

https://hg.mozilla.org/integration/mozilla-inbound/rev/4829b02a11af
Status: NEW → RESOLVED
Closed: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.