1569224 - Isolate Alt-Svc mappings for third-party tracker channels based on the origin of the top-level document

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, task)

Description

[(no longer active)]

Similar to bug 1535697, we'd like to isolate Alt-Svc mappings for third-party tracker channels based on the origin of the top-level document.

Comment 1

[Anne (:annevk)]

Do we need this if we do it always? I'd suggest we do the same thing as we do for the HTTP cache, which is to always key with the top-level origin as well.

Comment 2

[(no longer active)]

(In reply to Anne (:annevk) from comment #1)

Do we need this if we do it always? I'd suggest we do the same thing as we do for the HTTP cache, which is to always key with the top-level origin as well.

Yes, we should do this based on the cache isolation pref as well as ETP. By the time that we ship cache isolation we'd effectively do this "always", but before then we should do this for third-party trackers. It's not clear to me how long it'll take for us to ship cache isolation, hence this separate bug. The bug to do this always is bug 1553003.

(FWIW the code changes involved are mostly the same for these two cases anyway.)

Comment 3

[(no longer active)]

Attachment: Bug 1569224 - Part 1: Isolate Alt-Svc mappings for third-party tracker channels based on the origin of the top-level document;

Comment 4

[(no longer active)]

Attachment: Bug 1569224 - Part 2: Add a unit test for examining the alt-svc cache isolation for third-party trackers;

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/deb0da920913
Part 1: Isolate Alt-Svc mappings for third-party tracker channels based on the origin of the top-level document; r=michal
https://hg.mozilla.org/integration/autoland/rev/047e16b38566
Part 2: Add a unit test for examining the alt-svc cache isolation for third-party trackers; r=michal

Comment 6

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/deb0da920913
https://hg.mozilla.org/mozilla-central/rev/047e16b38566

Comment 7

[(no longer active)]

Steven, do you mind documenting this change for Firefox 70 please? Thanks!

Comment 8

[Steven Englehardt [:englehardt]]

Done. https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Privacy/Storage_access_policy$compare?locale=en-US&to=1581041&from=1572317