Closed Bug 913810 Opened 6 years ago Closed 6 years ago

HTTP cache v2: make nsHttpChannel properly react to FILE_NOT_FOUND from the cache

Categories

(Core :: Networking: Cache, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla31

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

(Whiteboard: [cache2])

Attachments

(1 file, 1 obsolete file)

Attached patch wip1 (obsolete) — Splinter Review
No description provided.
Depends on: 913812
Attached patch v1Splinter Review
- when cache load prematurely ends (in OnStartRequset, before listener is notified or delivered any data) with either FILE_CORRUPTED or NOT_FOUND, we doom the cache entry and reload (by an internal redirect to the same URI)
- this patch doesn't (and is not intended to) solve any problems found during 2nd and further chunk loads, a new bug may be filled for that (not needed for v1 enable)
- locally tested
Attachment #801125 - Attachment is obsolete: true
Attachment #8410962 - Flags: review?(michal.novotny)
Attachment #8410962 - Flags: review?(michal.novotny) → review+
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/e03b484bafc3
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.