Closed Bug 818616 Opened 12 years ago Closed 12 years ago

Perms Matrix vs. Installer - audio, system XHR, storage, cellbroadcast is undefined in the perms matrix

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jsmith, Assigned: sicking)

References

Details

Permissions installer states the following:

231                            audio: {
232                              app: DENY_ACTION,
233                              privileged: ALLOW_ACTION,
234                              certified: ALLOW_ACTION,
235                              channels: ["normal", "content", "notification",
236                                "alarm", "telephony", "ringer", "publicnotification"]
237                            },

However, nothing is called out about this in the perms matrix. What's the story here?
Flags: needinfo?(jonas)
Collapsing the bug per Jonas's feedback. Assigning to Gregor as he said he would handle these bugs.
Assignee: nobody → anygregor
Flags: needinfo?(jonas)
Summary: Perms Matrix vs. Installer - audio is undefined in the perms matrix → Perms Matrix vs. Installer - audio, system XHR, storage, cellbroadcast is undefined in the perms matrix
Summary: Perms Matrix vs. Installer - audio, system XHR, storage, cellbroadcast is undefined in the perms matrix → Perms Matrix vs. Installer - audio, system XHR, time, storage, cellbroadcast is undefined in the perms matrix
Looks like time is okay here. Removing that from title.
Summary: Perms Matrix vs. Installer - audio, system XHR, time, storage, cellbroadcast is undefined in the perms matrix → Perms Matrix vs. Installer - audio, system XHR, storage, cellbroadcast is undefined in the perms matrix
I'll take this one to update the permission matrix where needed and will file bugs on any changes that are needed.
Assignee: anygregor → jonas
Flags: needinfo?(jonas)
Summary: Perms Matrix vs. Installer - audio, system XHR, storage, cellbroadcast is undefined in the perms matrix → Perms Matrix vs. Installer - audio, system XHR, device-storage:apps, cellbroadcast is undefined in the perms matrix
Sorry, ignore my summary edit.
Summary: Perms Matrix vs. Installer - audio, system XHR, device-storage:apps, cellbroadcast is undefined in the perms matrix → Perms Matrix vs. Installer - audio, system XHR, storage, cellbroadcast is undefined in the perms matrix
We went over this today and it actually looks like the code is in very good shape.

The "audio" permission needs some updates in the code, Lucas will file a bug on that.
The "systemXHR" permission looks fine in both code and the permission matrix. The two are in sync and contain the right policy.
"storage" just needed updating in the permission matrix. Code is fine.
"cellbroadcast" just needed clarification in the permission matrix. Code is fine.
"device-storage:apps" is now updated in the permissions matrix but needs code changes. Bug 818583 is filed on that.
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: needinfo?(jonas)
Resolution: --- → FIXED
Ok so storage in the matrix doesnt match the installer, Ill raise a seperate bug.
You need to log in before you can comment on or make changes to this bug.