Closed
Bug 1649807
Opened 4 years ago
Closed 4 years ago
Consider ignoring Pragma header if Cache-Control is specified.
Categories
(Core :: Networking, task, P2)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla80
People
(Reporter: emilio, Assigned: emilio)
References
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
See bug 1599160 comment 43 and following.
|
Assignee | |
Comment 1•4 years ago
|
||
See bug 1599160 comment 43 for an example in the wild (though it has
been fixed since) which prevents us from caching resources
unnecessarily.
|
||
Updated•4 years ago
|
Assignee: nobody → emilio
Status: NEW → ASSIGNED
|
||
Updated•4 years ago
|
Severity: -- → S3
Priority: -- → P2
Whiteboard: [necko-triaged]
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/65e6c73de900 Ignore Pragma: no-cache if Cache-Control is specified. r=mayhemer,necko-reviewers
|
||
Comment 3•4 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox80:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
|
||
Comment 4•4 years ago
|
||
I am wandering how Chrome bahaves in this case? Emilio, have you tested it?
Flags: needinfo?(emilio)
|
|
Assignee | |
Comment 5•4 years ago
|
||
Yes, see https://phabricator.services.mozilla.com/D81879#2501609. I mentioned it there so that it got accounted for.
Flags: needinfo?(emilio)
|
||
Comment 6•4 years ago
|
||
| backout | ||
Per discussion with Emilio, this was backed out from Beta81 for causing bug 1664405. He's fixing that regression properly for 82+ with a patch in the bug.
https://hg.mozilla.org/releases/mozilla-beta/rev/8c2e83cd3507
status-firefox81:
--- → disabled
status-firefox82:
--- → fixed
|
|
Assignee | |
Updated•4 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•