If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

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

RESOLVED FIXED

Status

()

Core
Networking: Cache
--
minor
RESOLVED FIXED
4 years ago
11 months ago

People

(Reporter: mayhemer, Assigned: dragana)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
To makes things simpler, I vote for just dropping it completely.
(Reporter)

Updated

4 years ago
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

Comment 2

3 years ago
Where's the FTP stuff living?
(In reply to Florian Bender from comment #2)
> Where's the FTP stuff living?

netwerk/protocol/ftp
(Assignee)

Updated

3 years ago
Assignee: nobody → dd.mozilla
(Assignee)

Comment 4

3 years ago
Created attachment 8570192 [details] [diff] [review]
bug_913827_v1.patch
Attachment #8570192 - Flags: review?(michal.novotny)
(Assignee)

Comment 5

3 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=269a976094ad
Attachment #8570192 - Flags: review?(michal.novotny) → review+
(Assignee)

Updated

3 years ago
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
Last Resolved: 3 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.