[DeviceStorage] Add an extension for locked music files.

RESOLVED FIXED

Status

Firefox OS
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Harald, Assigned: dhylands)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Encrypted MP3 as new type generated by the Forward Lock app and played back by the patches Music app.

Via Fabrice: Gecko needs to know about mapping the extension to a mime type and the mime type needs to be added to known music files for device-storage.
(Reporter)

Updated

5 years ago
Blocks: 887071
This doesn't need to be security-sensitive
Group: core-security
Component: General → Gaia::Music
Sounds like we would need to modify toolkit/content/devicestorage.properties to include mp3x then.
Component: Gaia::Music → General
(Reporter)

Updated

5 years ago
Group: mozilla-corporation-confidential
As noted in the parent bug, I think we should use a different filename extension than "mp3x" because we might need to work with locked media formats that are not mp3s. But we can't just add ".fl" to the end of the filename because then we wouldn't be able to distinguish locked music from locked images (though I'm hoping we never have to deal with locked images".

A google search indicates that the extension ".lcka" is not in use. We could use that for any "locked audio".  Then, if necessary we have .lcki for images and .lckv for videos.
Editing the summary, nominating for 1.2, and assigning to Dave.
Assignee: nobody → dhylands
blocking-b2g: --- → koi?
Summary: Add .mp3x files to known file types for music → [DeviceStorage] Add an extension for locked music files.
(Assignee)

Comment 5

5 years ago
Created attachment 791015 [details] [diff] [review]
Add .lcka extension as a device storage audio extension
Attachment #791015 - Flags: review?(dflanagan)
Unchecking the confidential box. We can implement this in the open. If we need to discuss anything confidential, we can do that in the parent bug 887071.
Group: mozilla-corporation-confidential
Comment on attachment 791015 [details] [diff] [review]
Add .lcka extension as a device storage audio extension

Review of attachment 791015 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, Dave!
Attachment #791015 - Flags: review?(dflanagan) → review+
https://hg.mozilla.org/mozilla-central/rev/a4d600b4d8ba
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
blocking-b2g: koi? → ---
You need to log in before you can comment on or make changes to this bug.