[meta] Improvements to the Facebook shim script we wish to ship as part of phase 2.
Categories
(Core :: Privacy: Anti-Tracking, task, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox90 | --- | fixed |
People
(Reporter: twisniewski, Assigned: twisniewski)
References
Details
(Keywords: meta)
Attachments
(1 file)
As an addendum to our phase 2 shim effort, we'd like to ensure that the federated Facebook login shim works on as many known sites as possible.
This will require us to shim Facebook's impression.php tracker, as at least the site in bug 1657828 requires it to complete the login process. But it is not a script (it is a tracking pixel), so the shim code will not be able to match it without adjustments.
Similarly, we may want to render a stand-in button on elements with the CSS class fb-login-button, for those sites which rely on the FB SDK to render it. We may wish to do this on nightly-only, so we will need the shim to be able to know which release branch it is running under. I'd also like to begin testing opt-in placeholders for FB videos on the nightly cycle, so this fits.
In addition, there are several minor changes that will also help:
- allowlisting these URLs, which are required to complete the login on some sites, show profile images, etc:
*://graph.facebook.com/v*/*
*://graph.facebook.com/*/picture*
*://www.facebook.com/*/plugins/login_button.php*
*://www.facebook.com/x/oauth/status* - shimming
window.FB.uifor sites which use it instead ofwindow.FB.login. - shimming
window.FB.Event.(un)subscribefor sites which listen for login events.
Finally, there is an improvement we can make to the shim engine for opt-ins, which ensures that requests for opt-in content are allowed only on the tabs on which the user has opted in.
|
||
Updated•3 years ago
|
|
Assignee | |
Comment 1•3 years ago
|
||
Pushed by twisniewski@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8cb449906c81 Changes to the Facebook SmartBlock shim to bring it in line with phase 2 SmartBlock expectations, including relevant SmartBlock engine changes; r=denschub,webcompat-reviewers
|
||
Comment 3•3 years ago
|
||
| bugherder | ||
|
|
Assignee | |
Updated•3 years ago
|
Description
•