Isolate BlobURLs per agent-cluster
Categories
(Core :: Privacy: Anti-Tracking, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox81 | --- | fixed |
People
(Reporter: baku, Assigned: baku)
References
(Blocks 1 open bug, Regressed 1 open bug, )
Details
Attachments
(4 files)
BlobURLs can be used as a tracking vector. We want to limit them by agent-cluster.
|
Assignee | |
Comment 1•4 years ago
|
||
|
|
Assignee | |
Comment 2•4 years ago
|
||
Depends on D86914
|
|
Assignee | |
Comment 3•4 years ago
|
||
|
||
Comment 4•4 years ago
•
|
||
It isn't clear to me why we want this. I have similar questions what mkruisselbrink has.
I'm probably missing something obvious.
|
|
Assignee | |
Comment 5•4 years ago
|
||
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3609760c1e5d Isolate BlobURLs per agent-cluster, r=smaug https://hg.mozilla.org/integration/autoland/rev/a8620aecc29f Isolate BlobURLs per agent-cluster - tests, r=smaug https://hg.mozilla.org/integration/autoland/rev/20401a2adaa6 Isolate BlobURLs per agent-cluster - pref, r=smaug https://hg.mozilla.org/integration/autoland/rev/f69f80e5659f Isolate BlobURLs per agent-cluster - cleanup remove method, r=smaug
|
||
Comment 7•4 years ago
|
||
Backed out 4 changesets (Bug 1658878) for causing failures in browser_blobURLIsolation.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/72ef55c14a4c2167a35254020b4c1192334f1913
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=313035995&repo=autoland&lineNumber=8542
|
|
Assignee | |
Updated•4 years ago
|
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/16a33c7b4ff3 Isolate BlobURLs per agent-cluster, r=smaug https://hg.mozilla.org/integration/autoland/rev/33c4b81f99a1 Isolate BlobURLs per agent-cluster - tests, r=smaug https://hg.mozilla.org/integration/autoland/rev/a5af0e097560 Isolate BlobURLs per agent-cluster - pref, r=smaug https://hg.mozilla.org/integration/autoland/rev/0288ef9ab65b Isolate BlobURLs per agent-cluster - cleanup remove method, r=smaug
|
||
Comment 9•4 years ago
|
||
Backed out 4 changesets (bug 1658878) for blob related leaks.
Backout link: https://hg.mozilla.org/integration/autoland/rev/26091ecbe5db8031e056c220ef4e09c12df21209
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=313051685&repo=autoland&lineNumber=14827
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=313051685&repo=autoland&lineNumber=15759
[task 2020-08-14T19:25:08.129Z] 19:25:08 INFO - TEST-START | /encoding/legacy-mb-japanese/shift_jis/sjis-encode-form-errors-han.html?2001-3000
[task 2020-08-14T19:25:08.130Z] 19:25:08 INFO - Closing window 182
[task 2020-08-14T19:25:08.145Z] 19:25:08 INFO - Got chrome assert count 1
[task 2020-08-14T19:25:08.153Z] 19:25:08 INFO - TEST-UNEXPECTED-FAIL | /html/cross-origin-embedder-policy/blob.https.html | assertion count 1 is more than expected 0 assertions
[task 2020-08-14T19:25:08.154Z] 19:25:08 INFO - ......
[task 2020-08-14T19:25:08.155Z] 19:25:08 INFO - TEST-OK | /html/cross-origin-embedder-policy/blob.https.html | took 1621ms
...
[task 2020-08-14T19:26:27.836Z] 19:26:27 INFO - TEST-INFO | leakcheck | default leaked 2 nsSimpleURI
[task 2020-08-14T19:26:27.836Z] 19:26:27 INFO - TEST-INFO | leakcheck | default leaked 11 nsStandardURL
[task 2020-08-14T19:26:27.836Z] 19:26:27 INFO - TEST-INFO | leakcheck | default leaked 813 nsStringBuffer
[task 2020-08-14T19:26:27.837Z] 19:26:27 INFO - TEST-INFO | leakcheck | default leaked 41 nsTArray_base
[task 2020-08-14T19:26:27.837Z] 19:26:27 INFO - TEST-INFO | leakcheck | default leaked 1 nsThread
[task 2020-08-14T19:26:27.838Z] 19:26:27 INFO - TEST-INFO | leakcheck | default leaked 2 nsVariant
[task 2020-08-14T19:26:27.839Z] 19:26:27 INFO - TEST-UNEXPECTED-FAIL | leakcheck | default 21692 bytes leaked (BackstagePass, BlobURL, BlobURLInputStream, BrowsingContext, BrowsingContextGroup, ...)
[task 2020-08-14T19:26:27.839Z] 19:26:27 INFO -
[task 2020-08-14T19:26:27.840Z] 19:26:27 INFO - Closing logging queue
|
|
||
Comment 10•4 years ago
|
||
Also seeing the following starting with the backed out changes:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=313052624&repo=autoland&lineNumber=2465
|
|
Assignee | |
Updated•4 years ago
|
|
||
Comment 11•4 years ago
|
||
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/da55dec738c5 Isolate BlobURLs per agent-cluster, r=smaug https://hg.mozilla.org/integration/autoland/rev/703f9a9e5fde Isolate BlobURLs per agent-cluster - tests, r=smaug https://hg.mozilla.org/integration/autoland/rev/27d5d028b1af Isolate BlobURLs per agent-cluster - pref, r=smaug https://hg.mozilla.org/integration/autoland/rev/626c0384ddb8 Isolate BlobURLs per agent-cluster - cleanup remove method, r=smaug
|
|
||
Comment 12•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/da55dec738c5
https://hg.mozilla.org/mozilla-central/rev/703f9a9e5fde
https://hg.mozilla.org/mozilla-central/rev/27d5d028b1af
https://hg.mozilla.org/mozilla-central/rev/626c0384ddb8
|
||
Comment 13•4 years ago
|
||
:baku, according to mozregression, this seems to have broken the firefox screenshots feature which uses blob URIs for the screenshot/preview, see issue 5438. Do you have any suggestions on what we need to do to fix it?
|
|
||
Comment 14•4 years ago
|
||
(In reply to Sam Foster [:sfoster] (he/him) from comment #13)
:baku, according to mozregression, this seems to have broken the firefox screenshots feature which uses blob URIs for the screenshot/preview, see issue 5438. Do you have any suggestions on what we need to do to fix it?
I see this is already filed in bugzilla as bug 1660651. The need-info should stand though - any suggestions there appreciated.
|
|
Assignee | |
Comment 15•4 years ago
|
||
We have a separate bug for this regression. Let's move the discussion there.
Description
•