Discord snap `desktop` interface

store-requests privileged-interfaces
1

Hi folks

One of the snapcrafters has submitted a PR a while back that uses the desktop-file-ids attribute, which seems to have broken the auto-connect.

My understanding is that it shouldn’t. If we need a new auto-connect, could that please be granted to discord?

Failure can be seen here: Do not change desktop file name · snapcrafters/discord@b968cf5 · GitHub

Thanks! Jon

2

This request has been added to the queue for review by the @reviewers team.

1 Like
3

hi @jnsgruk - since the attribute desktop-file-ids is specified, it requires manual review. I don’t see an issue granting this to discord. +1 (#voteFor) from me

1 Like
4

Hey @jnsgruk

Thanks for bringing this up. So I can confirm that the failure is expected, as snaps using the desktop-file-ids are not allow to install unless explicitly approved via snap-declaration.

I think this request is reasonable. My only concern is that the desktop interface documentation states that the attribute should be a list of desktop file ids with valid D-Bus well-known names. From the dbus specification, Bus names must contain at least one '.' (period) character (and thus at least two elements). Thus, discord does not meet this criteria.

I think it might be an error in the interface documentation and what really needs to be a valid D-Bus well-known name is the compete file name (including the .desktop suffix) rather than the values specified in the desktop-file-ids attribute. However, I would like it to be confirmed by @pedronis before granting it.

Thanks

5

Yeh fair enough. I think for now we can leave it - we’ve worked around the problem by patching the Electron app to properly set the application name, and I think the original issue is now gone.

1 Like