Process fallbacks on out-of-domain redirects

RESOLVED FIXED

Status

()

defect
P3
normal
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: dcamp, Assigned: mayhemer)

Tracking

({fixed1.9.1})

unspecified
Points:
---
Bug Flags:
blocking1.9.1 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

According to the application cache spec:

--
Fetch the resource normally. If this results in a redirect to a resource with another origin (indicative of a captive portal), or a 4xx or 5xx status code or equivalent, or if there were network errors (but not if the user canceled the download), then instead get, from the cache, the resource of the fallback entry corresponding to the matched namespace. Abort these steps.
--

We currently don't handle the "redirect to a resource with another origin" case.
Flags: blocking1.9.1?
Dave, what's the result of us not handling this properly here? Trying to figure out whether this needs to block or not...
If someone is connected to a captive portal, and the offline app relies on fallbacks for functionality, the user will get captive portal content instead of the intended fallback.
Posted patch v1 (obsolete) — Splinter Review
Patch on top of patch from bug 460335.
Attachment #359271 - Flags: review?(dcamp)
Status: NEW → ASSIGNED
Flags: blocking1.9.1? → blocking1.9.1+
Priority: -- → P3
Fixed typo and perf hit.
Attachment #359271 - Attachment is obsolete: true
Attachment #359392 - Flags: review?(dcamp)
Attachment #359271 - Flags: review?(dcamp)
Dcamp?  Poke for review here.
Attachment #359392 - Flags: review?(dcamp) → review+
http://hg.mozilla.org/mozilla-central/rev/cd9c6a4a5e7f
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Attachment #359392 - Attachment description: v1.1 → v1.1 [Check-in comment 6]
Comment on attachment 359392 [details] [diff] [review]
v1.1 [Check-in comment 6][Checkin 1.9.1 comment 7]

http://hg.mozilla.org/releases/mozilla-1.9.1/rev/70b3cd0bc64f
Attachment #359392 - Attachment description: v1.1 [Check-in comment 6] → v1.1 [Check-in comment 6][Checkin 1.9.1 comment 7]
Keywords: fixed1.9.1
You need to log in before you can comment on or make changes to this bug.