Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Unnecesary sandbox hole for MPRIS #65

Open
A6GibKm opened this issue Mar 1, 2024 · 0 comments · May be fixed by #67
Open

Unnecesary sandbox hole for MPRIS #65

A6GibKm opened this issue Mar 1, 2024 · 0 comments · May be fixed by #67

Comments

@A6GibKm
Copy link
Contributor

A6GibKm commented Mar 1, 2024

The permission at https://github.com/flathub/com.vixalien.decibels/blob/master/com.vixalien.decibels.json#L21 is unnecessary, by default apps can serve MPRIS over org.mpris.MediaPlayer2.APP_ID , e.g. org.mpris.MediaPlayer2.com.vixalien.decibel.

Of course for this to work you would need to modify the MPRIS related code. See https://gitlab.gnome.org/World/podcasts/-/commit/026145e0c7354f5b9df43916168d42885a74d005 for example.
A6GibKm added a commit to A6GibKm/decibels that referenced this issue Mar 2, 2024
@A6GibKm
mpris: Set app id as identity
c37e8eb 
Flatpak automatically exposes org.mpris.MediaPlayer2.APP_ID as a owned
dbus name.

Fixes: vixalien#65
@A6GibKm A6GibKm linked a pull request Mar 2, 2024 that will close this issue
Open
A6GibKm added a commit to A6GibKm/decibels that referenced this issue Mar 2, 2024
@A6GibKm
mpris: Set app id as identity
ac8f29c 
Flatpak automatically exposes org.mpris.MediaPlayer2.APP_ID as a owned
dbus name.

Fixes: vixalien#65
A6GibKm added a commit to A6GibKm/decibels that referenced this issue Mar 11, 2024
@A6GibKm
mpris: Set app id as identity
aff3583 
Flatpak automatically exposes org.mpris.MediaPlayer2.APP_ID as a owned
dbus name.

Fixes: vixalien#65
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

mpris: Set app id as identity A6GibKm/decibels
1 participant
@A6GibKm