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

Flathub protal directory #57

Closed
Eonfge opened this issue Mar 15, 2022 · 3 comments · Fixed by #68
Closed

Flathub protal directory #57

Eonfge opened this issue Mar 15, 2022 · 3 comments · Fixed by #68
Assignees
Labels
bug Something isn't working flatpak

Comments

@Eonfge
Copy link

Eonfge commented Mar 15, 2022

Describe the bug
When you open an app with the Flatpak version of the App, you see the portal-location of the file and not the actual file location.

Screenshot from 2022-03-15 10-29-30

Expected behavior

To fix this, you should take a closer look at the NativeFileChooserApi spec. There is a specific method to get the actual location and not the security-portal symlink.

https://docs.gtk.org/gtk4/class.FileChooserNative.html

Environment (please complete the following information):

  • OS: Fedora Linux 35
  • Install method: Flathub
@Eonfge Eonfge added the bug Something isn't working label Mar 15, 2022
@GeopJr
Copy link
Owner

GeopJr commented Mar 15, 2022

Thanks for raising this issue!

Could you point me at that method? I can't seem to find it.
The closest I could find is flatpak/xdg-desktop-portal#475, which mentions adding host filesystem permission as a workaround. I'd like to avoid it unless there's no other solution.

@GeopJr GeopJr added the flatpak label Mar 15, 2022
@Eonfge
Copy link
Author

Eonfge commented Mar 16, 2022

The closest I could find is flatpak/xdg-desktop-portal#475, which mentions adding host filesystem permission as a workaround. I'd like to avoid it unless there's no other solution.

That should not be necessary, and it's also undesirable: An application like yours that should help people secure their computer, should not rely on such a brute-force method.

By the sound of it, showing the full path seems to be impossible right now. I didn't know that, so perhaps it's best to put it on the to-do list. For now, as a 'triage' you can just not display the file-path inside a flatpak. Bit blunt, but it solves the confusion it otherwise creates.

@GeopJr
Copy link
Owner

GeopJr commented Mar 16, 2022

That sounds like the best solution at the moment. I'll add a runtime check just in case someone gives it host filesystem permission manually. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working flatpak
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants