Expose xattr checking and setting functionality
Categories
(Toolkit Graveyard :: OS.File, enhancement, P1)
Tracking
(firefox-esr6069+ fixed, firefox-esr6869+ fixed, firefox68 wontfix, firefox69+ fixed, firefox70+ fixed)
People
(Reporter: bryce, Assigned: bryce)
References
Details
Attachments
(3 files)
47 bytes,
text/x-phabricator-request
|
ryanvm
:
approval-mozilla-beta+
jcristau
:
approval-mozilla-esr60+
jcristau
:
approval-mozilla-esr68+
|
Details | Review |
47 bytes,
text/x-phabricator-request
|
ryanvm
:
approval-mozilla-beta+
jcristau
:
approval-mozilla-esr60+
jcristau
:
approval-mozilla-esr68+
|
Details | Review |
47 bytes,
text/x-phabricator-request
|
ryanvm
:
approval-mozilla-beta+
jcristau
:
approval-mozilla-esr60+
jcristau
:
approval-mozilla-esr68+
|
Details | Review |
It would be useful to be able to check and set extended attributes/xattrs to address bug 1566523 (which is Mac specific). Beyond that bug such functionality seems like it may have a wider general use.
I believe this functionality would be specific to the Unix part of the lib, as I'm not sure Windows has a suitable analog.
Assignee | ||
Comment 1•6 years ago
|
||
Marking P1 since we want this functionality in 70 because of dependent bugs.
Bug 1566523 only requires the ability to remove an xattr, but adding the setting a checking functionality seems necessary for sane testing.
I've got a first draft of the removal piece that I will push to review shortly as I want to start iterating on feedback as I'm new to this area of the codebase. To be clear: I intend to follow up that patch with further functionality and tests and will not land that patch alone.
Assignee | ||
Comment 2•6 years ago
|
||
Add functionality to remove an extended attribute from a file. This
functionality is currently MacOS specific due to that being the only use case we
currently have (see bug 1566523).
Assignee | ||
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
Updated•6 years ago
|
Assignee | ||
Comment 5•6 years ago
|
||
Depends on D38523
Assignee | ||
Comment 6•6 years ago
|
||
Depends on D38964
Assignee | ||
Comment 7•6 years ago
|
||
Assignee | ||
Comment 8•6 years ago
|
||
Assignee | ||
Comment 9•6 years ago
|
||
[Tracking Requested - why for this release]: This bug will be required to have EME work on MacOS Catalina (it blocks bug 1566523 which in turn blocks our ability to use the CDM). Users are already encountering this is the Catalina beta and I would like to uplift if possible so the fix is available sooner.
Assignee | ||
Comment 10•6 years ago
|
||
[Tracking Requested - why for this release]: Please see comment 9.
Updated•6 years ago
|
Assignee | ||
Comment 11•6 years ago
|
||
Comment 12•6 years ago
|
||
Comment 13•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8ad6a2f6b8c7
https://hg.mozilla.org/mozilla-central/rev/a72f3a07f505
https://hg.mozilla.org/mozilla-central/rev/45483fab866c
Assignee | ||
Comment 14•6 years ago
|
||
Comment on attachment 9079091 [details]
Bug 1566700 - Add File.OS macRemoveXAttr. r?yoric!
Beta/Release Uplift Approval Request
- User impact if declined: Playback of premium video will not work under MacOS Catalina due to the CDM being blocked by new security features. These changes are needed to remove quarantine from the Widevine CDM once we extract it. If we do not do this loading of the CDM will be blocked by the OS.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: Bug 1566523, Bug 1558924, Bug 1566180
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The patches adds new functionality which is only currently used by bugs in this bug's dependency chain. Uplift of just these patches alone is essentially a no-op. I will note the risks of the other bugs in the chain on their respective uplift requests.
- String changes made/needed: None.
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Bustage of premium video playback on MacOS Catalina. We expect Catalina will release near the end of September, thought this date is not confirmed.
- User impact if declined: Playback of premium video will not work under MacOS Catalina due to the CDM being blocked by new security features. These changes are needed to remove quarantine from the Widevine CDM once we extract it. If we do not do this loading of the CDM will be blocked by the OS.
- Fix Landed on Version: 70
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The patches adds new functionality which is only currently used by bugs in this bug's dependency chain. Uplift of just these patches alone is essentially a no-op. I will note the risks of the other bugs in the chain on their respective uplift requests.
- String or UUID changes made by this patch: None.
Assignee | ||
Updated•6 years ago
|
Comment 15•6 years ago
|
||
Comment on attachment 9079091 [details]
Bug 1566700 - Add File.OS macRemoveXAttr. r?yoric!
Fixes for encrypted video playback for macOS 10.15. Approved for 69.0b10.
Updated•6 years ago
|
Updated•6 years ago
|
![]() |
||
Comment 16•6 years ago
|
||
bugherder uplift |
Comment 17•6 years ago
|
||
Comment on attachment 9079091 [details]
Bug 1566700 - Add File.OS macRemoveXAttr. r?yoric!
necessary change for macos 10.15 support, approved for 68.1 and 60.9
Updated•6 years ago
|
Updated•6 years ago
|
![]() |
||
Comment 18•6 years ago
|
||
bugherder uplift |
Assignee | ||
Comment 20•6 years ago
|
||
Try push with patches hand applied for ESR60: https://treeherder.mozilla.org/#/jobs?repo=try&revision=fb1c01a1745497254582a87e29db139db8e8c768
![]() |
||
Comment 21•6 years ago
|
||
bugherder uplift |
Updated•2 years ago
|
Description
•