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

Consider removing full filesystem access #108

Open
pabl0 opened this issue May 11, 2020 · 3 comments
Open

Consider removing full filesystem access #108

pabl0 opened this issue May 11, 2020 · 3 comments

Comments

@pabl0
Copy link

pabl0 commented May 11, 2020

Why there is need for --filesystem=host by default, especially since it looks like VLC supports portals and can play media without any direct access outside of the app sandbox? Is it needed for some "sidecar" files like subtitles?

@TingPing
Copy link
Member

The subtitle case indeed won't work automatically, though the user can usually manually load them.

@ePirat
Copy link

ePirat commented Sep 9, 2023

@pabl0 How would that work for command-line usage of VLC? Is Flatpak doing some magic there to look for paths in the command line?

Also what about writing files like screenshots? Would that "just work" or would the user need to first configure the path explicitly using a portal dialog for this to work?

@TingPing
Copy link
Member

TingPing commented Sep 12, 2023

@pabl0 How would that work for command-line usage of VLC? Is Flatpak doing some magic there to look for paths in the command line?

It isn't automatic, you have to do flatpak run $appid --file-forwarding @@ /path/to/a/file @@

Also what about writing files like screenshots? Would that "just work" or would the user need to first configure the path explicitly using a portal dialog for this to work?

The app has to open a save dialog each time.

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

No branches or pull requests

3 participants