Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix remote shares access with GVFS 1.48 #101

Closed
x80486 opened this issue Mar 29, 2021 · 9 comments · Fixed by #104
Closed

Fix remote shares access with GVFS 1.48 #101

x80486 opened this issue Mar 29, 2021 · 9 comments · Fixed by #104

Comments

@x80486
Copy link

x80486 commented Mar 29, 2021

Reference: flathub/flathub#2180

@hfiguiere
Copy link
Collaborator

How is it broken?

@hadess
Copy link

hadess commented Mar 30, 2021

It won't be able to talk to gvfsd, so everything that relies on gvfs won't work as expected.

@Jehan
Copy link
Collaborator

Jehan commented Mar 30, 2021

So all stuff like remote mounts and stuff like this, is that it?

From the linked report, all it takes is for us to add "--filesystem=xdg-run/gvfsd" to the list of permission, right? Then we'll have support for gvfs pre- and post-1.48?

@hfiguiere
Copy link
Collaborator

My question, is in which case does it fail? I couldn't find one, but it's totally possible that the problem is me here.

The original issue says not every app needs to do that:

If your application doesn't need to talk to gvfs via D-Bus, then it doesn't need access to xdg-run/gvfsd.

@hadess
Copy link

hadess commented Mar 30, 2021

So all stuff like remote mounts and stuff like this, is that it?

Yes, so pretty much required for an app like GIMP that doesn't use portals but wants to integrate into GNOME.

From the linked report, all it takes is for us to add "--filesystem=xdg-run/gvfsd" to the list of permission, right? Then we'll have support for gvfs pre- and post-1.48?

Yes.

The original issue says not every app needs to do that:

If your app doesn't talk to gvfs, then also remove all the org.gtk.vfs D-Bus holes that got punched.

In short, if you use a non-portal file chooser, then you want to punch all the holes necessary for gvfs so you can mount remote shares and disks.

@Jehan
Copy link
Collaborator

Jehan commented Mar 30, 2021

If your app doesn't talk to gvfs, then also remove all the org.gtk.vfs D-Bus holes that got punched.

Nono it does, since it was added as a response to someone asking for it: 8739c53 - issue #4372 in GIMP tracker

Anyway maybe we can't reproduce the issue if we don't use gvfs 1.48 yet. I assume it depends on the gvfs present on host-side, @hadess ? (here I have 1.46 on a Fedora 33)

@hadess
Copy link

hadess commented Mar 30, 2021

Anyway maybe we can't reproduce the issue if we don't use gvfs 1.48 yet. I assume it depends on the gvfs present on host-side, @hadess ? (here I have 1.46 on a Fedora 33)

It does, gvfs isn't installed inside the sandbox.

@Jehan
Copy link
Collaborator

Jehan commented Mar 30, 2021

Thanks @hadess.

@hfiguiere If you want to do your first direct commit, be my guest. 😉

@hfiguiere
Copy link
Collaborator

I see, given that I still run Fedora 33...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants