Closed Bug 421682 Opened 16 years ago Closed 16 years ago

Disallowing 3rd party cookies in iframes breaks iframe facebook apps

Categories

(Firefox :: Settings UI, defect)

defect
Not set
major

Tracking

()

VERIFIED DUPLICATE of bug 417800

People

(Reporter: jbellis+mozilla, Unassigned)

Details

User-Agent:       Opera/9.25 (Macintosh; Intel Mac OS X; U; en)
Build Identifier: FF3 beta3

Most iframe facebook apps (as opposed to fbml-based fb apps, which always go through fb servers) need to set cookies to remember which facebook user is logged in.

The discussion in https://bugzilla.mozilla.org/show_bug.cgi?id=417286 shows at least one developer being surprised by this.  "we do, indeed, block
cookies in iframes with third-party URLs. this means our third-party blocking
is good, perhaps too good, as it already stands."

Asking users to manually change security settings is a bar most won't bother jumping over.  That's not really an acceptable workaround.  A dialog along the lines of "this page is setting a third party cookie.  add to whitelist?" might be okay.

The original https://bugzilla.mozilla.org/show_bug.cgi?id=324397 issue where cookie-blocking policy was changed asks for examples of sites this would break.  Facebook is a fairly major example, even though iframe-based apps are a minority.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.



For comparison, the latest versions of IE, FF2, and Opera allow such cookies in iframes.  (In IE's case this requires setting a p3p header but this is a well-known and trivial workaround that can be implemented on the server.)
it's a useful datapoint to know that facebook apps can break with third party cookie blocking. as you say, this highlights the need for a good solution for sites that do break - but we shouldn't cripple this feature and compromise privacy as a result. (note that we've already reverted making this pref the default, exactly because of the fallout - bug 417800.) see also bug 419596.

Status: UNCONFIRMED → RESOLVED
Closed: 16 years ago
Resolution: --- → DUPLICATE
Ah, thanks for clarifying that.  I didn't notice from bug 417800 that the pref had been reverted since the summary was proposing something else.
Verified dup
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.