Request review for auto connections for interfaces -Personal files, System files & Hardware observe for MqttDesk snap

[emitorino]

Hey @newbee_snap,

@newbee_snap

We are using personal files to read and write at above location a file created by our app for licensing purpose. So we request here to auto-connect interface for personal files.

I assume there is a file created by mqttdesk in the $HOME/.local/share/data/bconf directory, am I correct? If that’s correct, please adjust your declaration to request write access to the specific file instead of the whole directory, and if it becomes clear that the file is owned by mqttdesk, then you should be able to get the enough votes for auto-connection, otherwise you probably will get only use. Also, did you try the suggestion provided by @alexmurray in your prior post? Maybe personal-files is not even needed at all:

Request for Strict confinement to classic confinement for MqttDesk

This snap does not meet the requirements for classic confinement and so this is not suitable in this case. However, as I said above, I don’t think you should even need personal-files if the code can respect $HOME, however if this is not the case then personal-files can be used for access to ~/.local/share/bconf if this is strictly necessary. Also a number of existing interfaces do provide some access to some of the paths under /sys/devices/virtual/dmi/id - can you please be more specific as to …

Regarding access to /etc/os-release, both @alexmurray in the previous post as well as @ogra here indicated:

@ogra

to see the file from the host in this case, you must use /var/lib/snapd/hostfs/etc/os-release

I see your snap is still referring /etc/os-release:

read-os-release:
    interface: system-files
    read:
      - /etc/os-release 

Can you please update it to read from /var/lib/snapd/hostfs/etc/os-release? Also, please name the interface reference in a way it properly describes the file being accessed, such as hostfs-etc-os-release

Regarding hardware-observe, I still see in your snapcraft:

read-proc-cpuinfo:
    interface: hardware-observe
    read:
      - /proc/cpuinfo

Did you see what @ogra also indicated in your previous post? Request for Strict confinement to classic confinement for MqttDesk - #31 by ogra

Please make the required changes and let us know so we can proceed with the voting process.

Thanks!